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

[DNM] Post-"ghost"-release hotfix workflow #8264

Draft
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

angusbayley
Copy link
Contributor

@angusbayley angusbayley commented Oct 31, 2024

⚠️ DO NOT MERGE - must be merged in sync with global hotfix process change

Ticket here

Create hotfix from any release

This modifies the existing create hotfix workflow to accept parameters for version and application. Specifying these values allows you to create a hotfix branch at the point of any previous release

Screenshot 2024-10-31 at 15 19 15
  • version is optional, and if unspecified defaults to "latest" - a special keyword that bypasses the tag finding process (see below), instead creating the hotfix branch from main just as the workflow did before this PR
  • application is mandatory and is implemented as a choice dropdown with LLM and LLD as the only two options

If version is specified it is reformatted into a tag using the specified application, e.g. version 2.91.0 + LLD will be reformatted into a the tag for the 2.91.0 release @ledgerhq/[email protected]. This is then used to checkout the repo prior to the remaining hotfix steps.

Edge case behaviour:

  • when someone deletes the default value in version and submits it as empty: it falls back to a ref of main (test run)
  • when version is a non-existent version: git fetch fails during the checkout action (test run).

Enforce patch version bumps in hotfix

The PR also restricts version bumps to patches, by enforcing a check in the prepare-hotfix workflow. This is the counterpart to #8244 which disallows patch versioning on any branch that is not hotfix.

Remove merge steps

The PR also removes workflow steps for merging the hotfix branch back into main and develop. To confirm, the new process is:

  1. make updates made on the hotfix branch
  2. publish packages, create production build from the hotfix branch
  3. (don't merge back into main or develop)

Copy link

vercel bot commented Oct 31, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
ledger-live-docs ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 5:47pm
ledger-live-github-bot ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 5:47pm
native-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 5:47pm
react-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 5:47pm
web-tools ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2024 5:47pm

@live-github-bot live-github-bot bot added the automation CI/CD stuff label Oct 31, 2024
@angusbayley angusbayley force-pushed the support/post-ghost-release-hotfix-workflow branch 3 times, most recently from 3e1e0dc to b7be5c7 Compare October 31, 2024 13:04
@angusbayley angusbayley force-pushed the support/post-ghost-release-hotfix-workflow branch from 977ded9 to 5559f90 Compare October 31, 2024 15:11
@angusbayley angusbayley changed the title [DNM] Support/post ghost release hotfix workflow [DNM] Post-"ghost"-release hotfix workflow Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation CI/CD stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant