Skip to content

ci: add golden files for runtime error codes#44677

Closed
AndrewKushnir wants to merge 1 commit intoangular:masterfrom
AndrewKushnir:goldens_for_errors
Closed

ci: add golden files for runtime error codes#44677
AndrewKushnir wants to merge 1 commit intoangular:masterfrom
AndrewKushnir:goldens_for_errors

Conversation

@AndrewKushnir
Copy link
Contributor

Runtime error codes in the Core, Common and Forms packages were not included into the public-api group reviews. This commit creates the necessary golden files to keep track of further changes in the runtime codes.

This is a followup from #44398 (comment).

PR Type

What kind of change does this PR introduce?

  • CI related changes

Does this PR introduce a breaking change?

  • Yes
  • No

Runtime error codes in the Core, Common and Forms packages were not included into the `public-api` group reviews. This commit creates the necessary golden files to keep track of further changes in the runtime codes.

This is a followup from angular#44398 (comment).
@AndrewKushnir AndrewKushnir added state: WIP area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release labels Jan 10, 2022
@ngbot ngbot bot modified the milestone: Backlog Jan 10, 2022
@AndrewKushnir AndrewKushnir added action: review The PR is still awaiting reviews from at least one requested reviewer and removed state: WIP labels Jan 10, 2022
@AndrewKushnir AndrewKushnir marked this pull request as ready for review January 10, 2022 23:09
@AndrewKushnir AndrewKushnir requested review from devversion and removed request for alxhub, atscott, jelbourn and jessicajaniuk January 10, 2022 23:10
Copy link
Contributor

@atscott atscott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewed-for: public-api

@pullapprove pullapprove bot requested a review from dylhunn January 10, 2022 23:11
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🍪

reviewed-for: public-api

Copy link
Contributor

@dylhunn dylhunn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewed-for: public-api

@AndrewKushnir AndrewKushnir 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 Jan 11, 2022
@AndrewKushnir AndrewKushnir removed the request for review from devversion January 11, 2022 23:23
@atscott
Copy link
Contributor

atscott commented Jan 12, 2022

This PR was merged into the repository by commit 57d0ca1.

atscott pushed a commit that referenced this pull request Jan 12, 2022
Runtime error codes in the Core, Common and Forms packages were not included into the `public-api` group reviews. This commit creates the necessary golden files to keep track of further changes in the runtime codes.

This is a followup from #44398 (comment).

PR Close #44677
@atscott atscott closed this in 57d0ca1 Jan 12, 2022
amitbeck pushed a commit to amitbeck/angular that referenced this pull request Jan 13, 2022
Runtime error codes in the Core, Common and Forms packages were not included into the `public-api` group reviews. This commit creates the necessary golden files to keep track of further changes in the runtime codes.

This is a followup from angular#44398 (comment).

PR Close angular#44677
@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 Feb 12, 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 area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants