Skip to content
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

fix: support latest-rc releases #78

Merged
merged 18 commits into from
Oct 26, 2021
Merged

fix: support latest-rc releases #78

merged 18 commits into from
Oct 26, 2021

Conversation

peternhale
Copy link
Contributor

@W-10008913@

  • Modifies test-and-release for main to publish latest-rc tag to npm and stable-rc channel to s3
  • When runn ing on main, the job wull stop for a manual "approval" step. This is a safeguard to prevent unwanted changes release w/latest-rc
  • Add cron job to run weekly on Wed to promote latest-rc and stable-rc to latest/stable for npm and s3 respectfully

mdonnalley
mdonnalley previously approved these changes Oct 18, 2021
scripts/promote-stable-rc-mac Outdated Show resolved Hide resolved
scripts/promote-stable-rc-tarballs Outdated Show resolved Hide resolved
scripts/promote-stable-rc-win Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
@peternhale
Copy link
Contributor Author

@RodEsp digging into pack-and-upload-tarballs I see the pack:verify is called for that step and by the time that tarballs are created, the npm release has already been done, so I assume that pinning deps is not necessary in the pack tarball steps.

I adjusted the job "requires" to jobs that actually run in the workflow. Good catches.

@peternhale peternhale requested a review from RodEsp October 25, 2021 21:11
RodEsp
RodEsp previously approved these changes Oct 25, 2021
@peternhale peternhale merged commit 526f503 into main Oct 26, 2021
@peternhale peternhale deleted the phale/W-10008913 branch October 26, 2021 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants