Skip to content

docs: update CPU profiling section#22279

Merged
filipesilva merged 1 commit into
angular:masterfrom
alan-agius4:doc-cpu-profile
Dec 6, 2021
Merged

docs: update CPU profiling section#22279
filipesilva merged 1 commit into
angular:masterfrom
alan-agius4:doc-cpu-profile

Conversation

@alan-agius4

@alan-agius4 alan-agius4 commented Dec 2, 2021

Copy link
Copy Markdown
Collaborator

This section is outdated.

Closes #20336

@google-cla google-cla Bot added the cla: yes label Dec 2, 2021
@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release and removed cla: yes labels Dec 2, 2021
@google-cla google-cla Bot added the cla: yes label Dec 2, 2021
@alan-agius4 alan-agius4 requested a review from dgp1130 December 2, 2021 15:53
Comment thread packages/angular/cli/README.md Outdated
Comment thread packages/angular/cli/README.md Outdated
Comment thread packages/angular/cli/README.md Outdated
Comment thread packages/angular/cli/README.md Outdated
Comment thread packages/angular/cli/README.md
@alan-agius4 alan-agius4 requested a review from dgp1130 December 2, 2021 18:43
This section is outdated.

Closes #20336
@alan-agius4

alan-agius4 commented Dec 3, 2021

Copy link
Copy Markdown
Collaborator Author

@dgp1130, I tried to find some easy to follow official docs on how to record and load a CPU profile, however, I wasn't able, hence for now since this section is pretty small I suggest that we keep it until there is a official resource to replace it.

@dgp1130

dgp1130 commented Dec 3, 2021

Copy link
Copy Markdown
Collaborator

@alan-agius4, sounds good. Thanks for refactoring this, I'm glad we're able to simplify the docs here and clean up some tech on the way.

@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Dec 3, 2021
@filipesilva filipesilva merged commit b06dd03 into angular:master Dec 6, 2021
@alan-agius4 alan-agius4 deleted the doc-cpu-profile branch December 6, 2021 15:05
@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 Jan 6, 2022
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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document NG_BUILD_PROFILING

3 participants