-
-
Notifications
You must be signed in to change notification settings - Fork 6
chore(deps): bump softprops/action-gh-release from 2.6.1 to 3.0.0 #327
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -71,7 +71,7 @@ | |
| } > release_notes.md | ||
|
|
||
| - name: Create GitHub Release | ||
| uses: softprops/action-gh-release@153bb8e04406b158c6c84fc1615b65b24149a1fe # v2 | ||
| uses: softprops/action-gh-release@b4309332981a82ec1c5618f44dd2e27cc8bfbfda # v2 | ||
|
Check warning on line 74 in .github/workflows/release-on-tag.yml
|
||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🟡 Both workflow files pin the action to SHA b4309332981a82ec1c5618f44dd2e27cc8bfbfda (the v3.0.0 release commit) but the trailing comment still reads Extended reasoning...What the bug is and how it manifests Both workflow files were updated by this PR to pin The specific code path that triggers it The stale comment appears in two places:
In both cases the diff shows the old line also had Why existing code doesn't prevent it Dependabot automates SHA updates but does not parse or update trailing version comments. There is no CI lint rule checking that the comment matches the actual version tag the SHA resolves to, so the mismatch goes undetected automatically. What the impact would be There is no runtime impact — GitHub Actions resolves the action by SHA, ignoring the comment entirely. The SHA How to fix it Change uses: softprops/action-gh-release@b4309332981a82ec1c5618f44dd2e27cc8bfbfda # v3Step-by-step proof
Note on the duplicate refutation One verifier argued bug_002 is a duplicate of bug_001. While bug_001's description mentions "both workflow files", the two occurrences are in separate files at separate lines and each requires its own edit. The synthesis agent correctly merged them into a single unified report covering both locations. |
||
| with: | ||
| tag_name: v${{ steps.version.outputs.version }} | ||
| name: v${{ steps.version.outputs.version }} | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -114,7 +114,7 @@ jobs: | |||||
| echo -e "$NOTES" > release_notes.md | ||||||
|
|
||||||
| - name: Create GitHub Release | ||||||
| uses: softprops/action-gh-release@153bb8e04406b158c6c84fc1615b65b24149a1fe # v2 | ||||||
| uses: softprops/action-gh-release@b4309332981a82ec1c5618f44dd2e27cc8bfbfda # v2 | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same issue as
Suggested change
Prompt To Fix With AIThis is a comment left during a code review.
Path: .github/workflows/release.yml
Line: 117
Comment:
**Outdated version comment**
Same issue as `release-on-tag.yml` — the commit hash resolves to v3.0.0, not v2.
```suggestion
uses: softprops/action-gh-release@b4309332981a82ec1c5618f44dd2e27cc8bfbfda # v3
```
How can I resolve this? If you propose a fix, please make it concise. |
||||||
| with: | ||||||
| tag_name: v${{ steps.version.outputs.version }} | ||||||
| name: v${{ steps.version.outputs.version }} | ||||||
|
|
||||||
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.
The pinned commit hash now points to v3.0.0, but the trailing comment still reads
# v2. This is misleading for anyone auditing the pinned hash.Prompt To Fix With AI