Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Excalibur Profiler Dev Build #2844

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

eonarheim
Copy link
Member

@eonarheim eonarheim commented Dec 10, 2023

===:clipboard: PR Checklist :clipboard:===

  • 馃搶 issue exists in github for these changes
  • 馃敩 existing tests still pass
  • 馃檲 code conforms to the style guide
  • 馃搻 new tests written and passing / old tests updated with new scenario(s)
  • 馃搫 changelog entry added (or not needed)

==================

image

This PR adds a new Excalibur Profiler that is meant to work with the new browser extension (not yet released) https://github.com/excaliburjs/excalibur-extension

  • Adds a new Profiler type that will work with flame charts
  • New decorator to easily profile potentially expensive Excalibur methods
  • Adds a new build mechanism that will strip out debug code (like the Profiler

Copy link

github-actions bot commented Feb 9, 2024

This PR hasn't had any recent activity lately and is being marked as stale automatically.

@github-actions github-actions bot added the stale This issue or PR has not had any activity recently label Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale This issue or PR has not had any activity recently
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant