Skip to content

ref: restore unpinned actions#4243

Merged
aldy505 merged 1 commit intomasterfrom
aldy505/ref/restore-unpinned-actions
Mar 25, 2026
Merged

ref: restore unpinned actions#4243
aldy505 merged 1 commit intomasterfrom
aldy505/ref/restore-unpinned-actions

Conversation

@aldy505
Copy link
Collaborator

@aldy505 aldy505 commented Mar 25, 2026

Previously we have pinned actions for those created by 'getsentry/', now it's a maintenance burden to approve & merge every dependabot PR. Let's just unpin them. Therefore making Burak's life easier too.

Previously we have pinned actions for those created by 'getsentry/', now it's a maintenance burden to approve & merge every dependabot PR. Let's just unpin them. Therefore making Burak's life easier too.
@github-actions
Copy link

github-actions bot commented Mar 25, 2026

Changelog Preview

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Remove 'vroom-cleanup' container by aldy505 in #4217

Internal Changes 🔧

Deps

  • Bump getsentry/craft from 2.23.2 to 2.24.1 by dependabot in #4221
  • Bump astral-sh/setup-uv from 7.2.1 to 7.5.0 by dependabot in #4220

Other

  • Restore unpinned actions by aldy505 in #4243
  • Swap pre-commit with prek by aldy505 in #4235

🤖 This preview updates automatically when you update the PR.

@aldy505 aldy505 requested a review from BYK March 25, 2026 14:46
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Mutable tag on pull_request_target with secrets: inherit
    • I verified the risk is real and fixed it by repinning both privileged getsentry/craft workflow references from mutable @v2 to the immutable commit SHA f4889d04564e47311038ecb6b910fef6b6cf1363.

Create PR

Or push these changes by commenting:

@cursor push 0c26f0c466
Preview (0c26f0c466)
diff --git a/.github/workflows/changelog-preview.yml b/.github/workflows/changelog-preview.yml
--- a/.github/workflows/changelog-preview.yml
+++ b/.github/workflows/changelog-preview.yml
@@ -15,5 +15,5 @@
 
 jobs:
   changelog-preview:
-    uses: getsentry/craft/.github/workflows/changelog-preview.yml@v2
+    uses: getsentry/craft/.github/workflows/changelog-preview.yml@f4889d04564e47311038ecb6b910fef6b6cf1363 # 2.25.0
     secrets: inherit

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -33,7 +33,7 @@
           fetch-depth: 0
       - name: Prepare release
         id: prepare-release
-        uses: getsentry/craft@v2
+        uses: getsentry/craft@f4889d04564e47311038ecb6b910fef6b6cf1363 # 2.25.0
         env:
           GITHUB_TOKEN: ${{ steps.token.outputs.token }}
         with:

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

@aldy505 aldy505 merged commit e04fa72 into master Mar 25, 2026
19 checks passed
@aldy505 aldy505 deleted the aldy505/ref/restore-unpinned-actions branch March 25, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants