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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move experimental releases to a manually triggered workflow #9082

Merged
merged 1 commit into from Mar 19, 2024

Conversation

brophdawg11
Copy link
Contributor

@brophdawg11 brophdawg11 commented Mar 19, 2024

POC to streamline our experimental release process to run it manually from Github Actions.

Today, we have to:

  • checkout a local branch with a specific name
  • run yarn version:experimental
  • push branch and tags to github to trigger a CI workflow base don the specific tag name

With this change we would:

  • Push our feature branch we want to release from
  • Run this workflow in Github Actions with our feature branch name as an input

The branching and versioning will be subsumed by the action - so less manual stuff to run and we'd be able to delete some of the forks in our version.js and publish.js script if this pans out. Plus it would align experimental and nightly releases so we don't have 3 separate ways of releasing anymore (stable, nightly, experimental).

I'd like to merge this and give it a test run, and if it works we can move forward with trimming down scripts/ and such. If it doesn't work, we'll revert and pretend it never happened 馃檲

Copy link

changeset-bot bot commented Mar 19, 2024

鈿狅笍 No Changeset found

Latest commit: bf6b26b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

SHA=$(git rev-parse HEAD)
# get first 7 characters of sha
SHORT_SHA=${SHA::7}
SHORT_SHA=$(git rev-parse --short HEAD)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Slightly easier way to do this stolen from our version.js script - results in a 9 char SHA instead of 7.

workflow_dispatch:
inputs:
branch:
required: true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can't run this without specifying the branch to checkout and build from

@@ -0,0 +1,59 @@
name: 馃И Experimental Release
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a copy of the nightly.yml file with required changes for experimentals

@brophdawg11 brophdawg11 merged commit e37deeb into main Mar 19, 2024
5 checks passed
@brophdawg11 brophdawg11 deleted the brophdawg11/experimental-release-workflow branch March 19, 2024 14:32
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.

None yet

1 participant