Skip to content

chore: pin GitHub Actions to full-length commit SHAs#5760

Open
joshuarli wants to merge 1 commit intomasterfrom
pin-gha-actions
Open

chore: pin GitHub Actions to full-length commit SHAs#5760
joshuarli wants to merge 1 commit intomasterfrom
pin-gha-actions

Conversation

@joshuarli
Copy link
Copy Markdown
Member

Summary

  • Pin all GitHub Actions references in .github/ workflow files to full-length commit SHAs

Generated by devenv pin_gha.

🤖 Generated with Claude Code

@joshuarli joshuarli requested a review from a team as a code owner March 24, 2026 06:40
@joshuarli joshuarli requested a review from geoffg-sentry March 24, 2026 06:40
Copy link
Copy Markdown

@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 is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

steps:
- name: Run Sentry self-hosted e2e CI
uses: getsentry/self-hosted@master
uses: getsentry/self-hosted@871c182cb0a99dc1fad72cc7ce7889b514b0c5f0 # master
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hardcoded path breaks after SHA pinning of self-hosted action

Medium Severity

Pinning getsentry/self-hosted from @master to @871c182cb0a99dc1fad72cc7ce7889b514b0c5f0 changes the directory where GitHub Actions stores the downloaded action from .../self-hosted/master to .../self-hosted/871c182cb0a99dc1fad72cc7ce7889b514b0c5f0. The Inspect failure step on line 899 still hardcodes cd /home/runner/work/_actions/getsentry/self-hosted/master, which will now fail with "no such file or directory," preventing collection of debugging output when the e2e tests fail.

Fix in Cursor Fix in Web

Copy link
Copy Markdown

@geoffg-sentry geoffg-sentry left a comment

Choose a reason for hiding this comment

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

cursor makes a point worth reviewing for self-hosted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants