Skip to content

feat(core): introduce new DI profiling event#60158

Closed
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:di_profiler_new_event
Closed

feat(core): introduce new DI profiling event#60158
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:di_profiler_new_event

Conversation

@pkozlowski-opensource
Copy link
Copy Markdown
Member

This change introduces a new DI profiler event:
InjectorToCreateInstanceEvent. This new event allows us to measure DI tokens instantiation time.

@angular-robot angular-robot bot added detected: feature PR contains a feature commit area: core Issues related to the framework runtime labels Feb 28, 2025
@pkozlowski-opensource pkozlowski-opensource added target: minor This PR is targeted for the next minor release and removed area: core Issues related to the framework runtime detected: feature PR contains a feature commit labels Feb 28, 2025
@ngbot ngbot bot modified the milestone: Backlog Feb 28, 2025
@pkozlowski-opensource pkozlowski-opensource added area: core Issues related to the framework runtime detected: feature PR contains a feature commit labels Feb 28, 2025
@ngbot ngbot bot modified the milestone: Backlog Feb 28, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the di_profiler_new_event branch 2 times, most recently from 8f190e2 to b2e520c Compare March 3, 2025 10:16
This change introduces a new DI profiler event:
InjectorToCreateInstanceEvent. This new event allows
us to measure DI tokens instantiation time.
@pkozlowski-opensource pkozlowski-opensource added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 3, 2025
@pkozlowski-opensource pkozlowski-opensource marked this pull request as ready for review March 3, 2025 13:57
@AleksanderBodurri AleksanderBodurri added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 3, 2025
@mmalerba
Copy link
Copy Markdown
Contributor

mmalerba commented Mar 3, 2025

This PR was merged into the repository by commit 809b5b4.

The changes were merged into the following branches: main

@mmalerba mmalerba closed this in 809b5b4 Mar 3, 2025
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants