-
Notifications
You must be signed in to change notification settings - Fork 6.8k
build: update shared dev-infra package #23427
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: update shared dev-infra package #23427
Conversation
crisbeto
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
josephperrott
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Updates the shared dev-infra package to take advanatge of recent improvements for the release tool. This also fixes the issue where the merge tooling requires the `breaking change` label, while there also is the `state: breaking change` label. The ng-dev tool no longer accepts custom configuration of that label. Also the merge configuration no longer accepts custom labels, in order to achieve consistent labeling as specified in the Angular versioning/branching/labeling spec.
Updates the Github actions to latest revisions. The github actions now correctly deal with `state: breaking change` label.
Checks the `.ng-dev` tool configuration in CI, compared to doing it locally when the caretaker intends to perform a merge and then realizes the config is broken.
92e8dd3 to
1a3ea7b
Compare
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
See individual commits.