Skip to content

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

Merged
merged 3 commits into from
Aug 11, 2021

Conversation

devversion
Copy link
Member

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)

@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Aug 9, 2021
@devversion devversion force-pushed the release-tool-shared branch 2 times, most recently from 18e85e2 to ebc0636 Compare August 9, 2021 21:40
@devversion devversion added merge safe target: patch This PR is targeted for the next patch release labels Aug 9, 2021
@devversion devversion marked this pull request as ready for review August 9, 2021 21:46
@devversion devversion requested a review from a team as a code owner August 9, 2021 21:46
@devversion devversion force-pushed the release-tool-shared branch from ebc0636 to f203867 Compare August 10, 2021 09:18
@devversion devversion added target: major This PR is targeted for the next major release and removed target: patch This PR is targeted for the next patch release labels Aug 10, 2021
@devversion
Copy link
Member Author

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"™️ 😄

Copy link
Member

@crisbeto crisbeto left a 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
@devversion devversion force-pushed the release-tool-shared branch from c466109 to cdd6699 Compare August 10, 2021 20:08
@devversion devversion requested review from crisbeto and josephperrott and removed request for crisbeto August 10, 2021 20:08
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

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

LGTM

@josephperrott
Copy link
Member

@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?

@devversion
Copy link
Member Author

@josephperrott yeah, that's the plan. Since some new packages are not in patch, I will send a separate PR.

@devversion devversion added the action: merge The PR is ready for merge by the caretaker label Aug 10, 2021
@andrewseguin andrewseguin merged commit 0aec45a into angular:master Aug 11, 2021
devversion added a commit to devversion/material2 that referenced this pull request Aug 11, 2021
* 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
andrewseguin pushed a commit that referenced this pull request Aug 11, 2021
* 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
@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 Sep 11, 2021
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 cla: yes PR author has agreed to Google's Contributor License Agreement target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants