-
Notifications
You must be signed in to change notification settings - Fork 6.8k
build: switch to release tool from dev-infra #23333
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
Conversation
18e85e2
to
ebc0636
Compare
ebc0636
to
f203867
Compare
I've tested various scenarios of the tool in my forks (this took most of the time; the tool is difficult to test). Everything worked well based on the things I could test. Obviously the testing does not include the Wombat registry so that is kind of unknown to me. Other than that, "It should work"™️ 😄 |
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
Switches to the release tool from the shared dev-infra package. This script is more aligned with the versioning and branching the whole Angular organization follows as this point. We work around a couple of APIs that need to be discussed within the release tool in order to wire up custom release stage / publish validations. We still want to keep them for now so we use some monkey-patching (which is acceptable for the interim until we have determined which checks should be built-in / how consumers can wire up additional checks)
Remove wombat registry explicit entries
Address feedback
c466109
to
cdd6699
Compare
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
@devversion Do we need to create a patch version of this to allow for the tool to know how to build the patch releases going forward? |
@josephperrott yeah, that's the plan. Since some new packages are not in patch, I will send a separate PR. |
* build: switch to release tool from dev-infra Switches to the release tool from the shared dev-infra package. This script is more aligned with the versioning and branching the whole Angular organization follows as this point. We work around a couple of APIs that need to be discussed within the release tool in order to wire up custom release stage / publish validations. We still want to keep them for now so we use some monkey-patching (which is acceptable for the interim until we have determined which checks should be built-in / how consumers can wire up additional checks) * fixup! build: switch to release tool from dev-infra Remove wombat registry explicit entries * fixup! build: switch to release tool from dev-infra Address feedback
* build: switch to release tool from dev-infra Switches to the release tool from the shared dev-infra package. This script is more aligned with the versioning and branching the whole Angular organization follows as this point. We work around a couple of APIs that need to be discussed within the release tool in order to wire up custom release stage / publish validations. We still want to keep them for now so we use some monkey-patching (which is acceptable for the interim until we have determined which checks should be built-in / how consumers can wire up additional checks) * fixup! build: switch to release tool from dev-infra Remove wombat registry explicit entries * fixup! build: switch to release tool from dev-infra Address feedback
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. |
Switches to the release tool from the shared dev-infra
package. This script is more aligned with the versioning
and branching the whole Angular organization follows as this
point.
We work around a couple of APIs that need to be discussed
within the release tool in order to wire up custom release
stage / publish validations. We still want to keep them for
now so we use some monkey-patching (which is acceptable for
the interim until we have determined which checks should be
built-in / how consumers can wire up additional checks)