Skip to content

Danger: block manual edits to generated error codes#6996

Merged
AlvaroBrey merged 1 commit into
mainfrom
danger-generated-guard
Jun 15, 2026
Merged

Danger: block manual edits to generated error codes#6996
AlvaroBrey merged 1 commit into
mainfrom
danger-generated-guard

Conversation

@AlvaroBrey

@AlvaroBrey AlvaroBrey commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
CI and Danger policy only; no runtime SDK or app behavior changes.

Overview
Adds a Danger guard so PRs that touch Sources/Generated/ fail CI, steering error-code updates through the purchases-error-codes / update-error-codes automation instead of hand edits (via shared fail_on_generated_edits from RevenueCat/Dangerfile).

Bumps the CircleCI revenuecat/sdks-common-config orb 3.21.0 → 3.21.1 so automated error-code PRs get the expected labels from the shared orb.

Reviewed by Cursor Bugbot for commit a153497. Bugbot is set up for automated code reviews on this repo. Configure here.

@AlvaroBrey AlvaroBrey self-assigned this Jun 15, 2026
@AlvaroBrey AlvaroBrey force-pushed the danger-generated-guard branch from a48aaff to 5314050 Compare June 15, 2026 09:56
Call the shared fail_on_generated_edits helper for Sources/Generated/ (bypass via
pr:auto_codegen). Bump orb to 3.21.1, which applies the pr:auto_codegen label.
@AlvaroBrey AlvaroBrey force-pushed the danger-generated-guard branch from 5314050 to a153497 Compare June 15, 2026 10:00
@AlvaroBrey AlvaroBrey marked this pull request as ready for review June 15, 2026 10:12
@AlvaroBrey AlvaroBrey requested a review from a team as a code owner June 15, 2026 10:12
@AlvaroBrey AlvaroBrey requested review from a team and removed request for a team June 15, 2026 10:12
@AlvaroBrey AlvaroBrey merged commit 441c044 into main Jun 15, 2026
18 of 20 checks passed
@AlvaroBrey AlvaroBrey deleted the danger-generated-guard branch June 15, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants