Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Output migration stats #58321

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

Output migration stats #58321

wants to merge 3 commits into from

Conversation

pkozlowski-opensource
Copy link
Member

No description provided.

@pkozlowski-opensource
feat(migrations): capture output migration stats
e69377a 
This change captures different statistics from the run of the
output migration:
- total number of outputs detected
- number of problematic outputs that can't be migrated
- overall success rate
@pullapprove pullapprove bot requested a review from crisbeto October 23, 2024 10:25
@angular-robot angular-robot bot added detected: feature PR contains a feature commit area: migrations Issues related to `ng update` migrations labels Oct 23, 2024
@ngbot ngbot bot added this to the Backlog milestone Oct 23, 2024
@pkozlowski-opensource pkozlowski-opensource added action: review The PR is still awaiting reviews from at least one requested reviewer target: minor This PR is targeted for the next minor release labels Oct 23, 2024
@pkozlowski-opensource pkozlowski-opensource requested review from devversion and removed request for crisbeto October 23, 2024 10:27
devversion
devversion approved these changes Oct 23, 2024
View reviewed changes
@pkozlowski-opensource pkozlowski-opensource 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 Oct 23, 2024
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit cf70d62.

The changes were merged into the following branches: main

AndrewKushnir pushed a commit that referenced this pull request Oct 23, 2024
@pkozlowski-opensource @AndrewKushnir
feat(migrations): print output migration stats in ng generate (#58321)
cf70d62 
Propagates output migration stats to the ng generate schematic.

PR Close #58321
@angular-automatic-lock-bot
Copy link

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 Nov 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@devversion devversion devversion approved these changes

Assignees
No one assigned
Labels
action: merge The PR is ready for merge by the caretaker area: migrations Issues related to `ng update` migrations detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release
Projects
None yet
Milestone
v19 Feature Freeze Candidates
Development

Successfully merging this pull request may close these issues.
3 participants
@pkozlowski-opensource @AndrewKushnir @devversion