-
-
Notifications
You must be signed in to change notification settings - Fork 279
ci(release): Switch from action-prepare-release to Craft #3440
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
Open
BYK
wants to merge
51
commits into
main
Choose a base branch
from
ci/migrate-to-craft-action
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+25
−3
Open
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
ee4e7c7
ci(release): Switch from action-prepare-release to Craft
BYK 486e197
ci(release): Restore GitHub App token authentication
BYK 915ad33
fix: Pin actions to SHA and add permissions blocks
BYK da9b49f
fix: Use correct action version SHAs (restore original versions)
BYK 94ea76f
fix: Use correct action version SHAs (restore original versions)
BYK 5e3c97a
fix: Clean up action version comments
BYK c10928f
Update Craft SHA to 1c58bfd57bfd6a967b6f3fc92bead2c42ee698ce
BYK 6cca9f5
Add explicit permissions block to analyze.yml
BYK cb7a3c0
Add explicit permissions block to dart.yml
BYK 93d76a3
Add explicit permissions block to dio.yml
BYK 068fb81
Add explicit permissions block to drift.yml
BYK 49523f4
Add explicit permissions block to e2e_dart.yml
BYK 46d0cda
Add explicit permissions block to file.yml
BYK 51c28f0
Add explicit permissions block to firebase_remote_config.yml
BYK d98598f
Add explicit permissions block to flutter.yml
BYK 5913dd1
Add explicit permissions block to flutter_test.yml
BYK c4b6e0f
Add explicit permissions block to format-and-fix.yml
BYK 601ef8d
Add explicit permissions block to hive.yml
BYK 3ab8c18
Add explicit permissions block to isar.yml
BYK 0729237
Add explicit permissions block to link.yml
BYK c0e9dc3
Add explicit permissions block to logging.yml
BYK 928e9a5
Add explicit permissions block to metrics.yml
BYK bd2aa0f
Add explicit permissions block to min_version_test.yml
BYK fac7650
Add explicit permissions block to sqflite.yml
BYK 556682f
Add explicit permissions block to supabase.yml
BYK c67b543
Add explicit permissions block to testflight.yml
BYK 0c213db
Add explicit permissions block to web-example-ghpages.yml
BYK 493c0b6
Revert permissions changes to analyze.yml
BYK 1460e67
Revert permissions changes to dart.yml
BYK 65e1708
Revert permissions changes to dio.yml
BYK 4f5ca5f
Revert permissions changes to drift.yml
BYK a9fb284
Revert permissions changes to e2e_dart.yml
BYK df7d5e9
Revert permissions changes to file.yml
BYK 97e42a9
Revert permissions changes to firebase_remote_config.yml
BYK 9790035
Revert permissions changes to flutter.yml
BYK fce1b14
Revert permissions changes to flutter_test.yml
BYK 51467ab
Revert permissions changes to format-and-fix.yml
BYK 099b4fc
Revert permissions changes to hive.yml
BYK 88598fc
Revert permissions changes to isar.yml
BYK 4a85db8
Revert permissions changes to link.yml
BYK 7869a62
Revert permissions changes to logging.yml
BYK 6a91a9b
Revert permissions changes to metrics.yml
BYK c75f162
Revert permissions changes to min_version_test.yml
BYK a450fb7
Revert permissions changes to sqflite.yml
BYK 6daf668
Revert permissions changes to supabase.yml
BYK db8876c
Revert permissions changes to testflight.yml
BYK a6a4059
Revert permissions changes to web-example-ghpages.yml
BYK e2641dc
fix: revert extraneous changes to non-release workflow files
BYK 4e1e223
fix: clean up release.yml formatting and version comments
BYK 9c0df90
build(craft): Update Craft action to c6e2f04
BYK 37107ff
chore: add unlabeled trigger to changelog-preview
BYK File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| name: Changelog Preview | ||
| on: | ||
| pull_request: | ||
| types: | ||
| - opened | ||
| - synchronize | ||
| - reopened | ||
| - edited | ||
| - labeled | ||
| - unlabeled | ||
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
|
|
||
| jobs: | ||
| changelog-preview: | ||
| uses: getsentry/craft/.github/workflows/changelog-preview.yml@v2 | ||
| secrets: inherit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: The release workflow will fail if the optional
versioninput is not provided, as it passes an empty string to Craft which causes a validation error.Severity: CRITICAL
Suggested Fix
To align with the intended behavior of defaulting to auto-versioning, modify the
versionparameter to handle an empty string. Change the line toversion: ${{ github.event.inputs.version || 'auto' }}. This will ensure that if no version is supplied, 'auto' is passed to Craft instead of an empty string.Prompt for AI Agent
Did we get this right? 👍 / 👎 to inform future reviews.