Skip to content

fix(warden): Disable Warden if secrets are not available#178

Open
DragoonAethis wants to merge 1 commit intogetsentry:mainfrom
DragoonAethis:no-warden-without-secrets
Open

fix(warden): Disable Warden if secrets are not available#178
DragoonAethis wants to merge 1 commit intogetsentry:mainfrom
DragoonAethis:no-warden-without-secrets

Conversation

@DragoonAethis
Copy link
Copy Markdown

For contributors outside the getsentry org, secrets are not available, which causes this job to fail. Disable it for now to prevent false positives.

(I think this repo is missing the legal boilerplate etc by default? Either way, copied over from another PR:)

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

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.

Reviewed by Cursor Bugbot for commit 8544c51. Configure here.

Comment thread .github/workflows/warden.yml Outdated
For contributors outside the getsentry org, secrets are not available,
which causes this job to fail. Disable it for now.
@DragoonAethis DragoonAethis force-pushed the no-warden-without-secrets branch from 8544c51 to c647203 Compare April 29, 2026 01:22
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.

1 participant