ci(release): Switch from action-prepare-release to Craft#3069
ci(release): Switch from action-prepare-release to Craft#3069szokeasaurusrex merged 24 commits intomasterfrom
Conversation
This PR migrates from the deprecated action-prepare-release to the new Craft GitHub Actions (reusable workflow or composite action). Changes: - Migrate .github/workflows/release.yml to Craft reusable workflow
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. Build / dependencies / internal 🔧
Other
🤖 This preview updates automatically when you update the PR. |
The previous migration incorrectly removed the GitHub App token authentication step. This commit restores it by switching to the composite action pattern which preserves the auth flow.
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
| permissions: | ||
| contents: write | ||
| pull-requests: write |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
szokeasaurusrex
left a comment
szokeasaurusrex
left a comment
There was a problem hiding this comment.
We already discussed this over Slack, but please undo these extraneous version comments 🙏 then I will properly review this 😄
- Remove changelog-preview.yml as requested by reviewer - Restore original version comments (remove duplicate # vX # X.Y.Z) - Restore original formatting and runner in release.yml - Remove unnecessary permissions from GHCR workflows (keep only packages: write)
| version: | ||
| description: Version to release | ||
| required: true | ||
| description: Version to release (or "auto") | ||
| required: false |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
.github/workflows/release.yml
Outdated
| version: ${{ github.event.inputs.version }} | ||
| force: ${{ github.event.inputs.force }} | ||
| merge_target: ${{ github.event.inputs.merge_target }} | ||
| version: ${{ inputs.version }} |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
szokeasaurusrex
left a comment
szokeasaurusrex
left a comment
There was a problem hiding this comment.
made a whitespace and comment change, I hope that is ok!
|
@szokeasaurusrex if you set the comment to a specific version like |
|
@BYK ive got an action that bumps all the shas and the version numbers every week |
2 participants
Summary
This PR migrates from the deprecated
action-prepare-releaseto the new Craft GitHub Actions.Changes
.github/workflows/release.ymlto Craft reusable workflowDocumentation
See https://getsentry.github.io/craft/github-actions/ for more information.