Skip to content

chore: Add PR validation workflow#5239

Merged
stephanie-anderson merged 1 commit intomainfrom
chore/add-validate-pr-workflow
Mar 27, 2026
Merged

chore: Add PR validation workflow#5239
stephanie-anderson merged 1 commit intomainfrom
chore/add-validate-pr-workflow

Conversation

@stephanie-anderson
Copy link
Copy Markdown
Contributor

@stephanie-anderson stephanie-anderson commented Mar 27, 2026

Summary

  • Adds a validate-pr.yml workflow to automatically validate non-maintainer PRs
  • Checks that PRs reference a GitHub issue with prior discussion between the author and a maintainer
  • Closes PRs that don't meet contribution guidelines (no issue reference, no maintainer discussion, or issue assigned to someone else)
  • Enforces that all PRs start as drafts

Rollout of getsentry/sentry-python#4233 across all SDK repos.

#skip-changelog

Co-Authored-By: Claude Opus 4.6 (1M context) [email protected]

Automatically validates non-maintainer PRs by checking:
- Issue reference exists in PR body
- Referenced issue has discussion between author and maintainer
- Referenced issue is not assigned to someone else

Also enforces that all PRs start as drafts.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

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


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 27, 2026

Sentry Build Distribution

App Name App ID Version Configuration Install Page
SDK Size io.sentry.tests.size 8.37.1 (1) release Install Build

@github-actions
Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 357.22 ms 424.10 ms 66.88 ms
Size 0 B 0 B 0 B

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
6405ec5 310.88 ms 354.56 ms 43.69 ms
d5a29b6 298.62 ms 391.78 ms 93.16 ms
f064536 329.00 ms 395.62 ms 66.62 ms
fcec2f2 357.47 ms 447.32 ms 89.85 ms
bbc35bb 324.88 ms 425.73 ms 100.85 ms
694d587 312.37 ms 402.77 ms 90.41 ms
b8bd880 314.56 ms 336.50 ms 21.94 ms
889ecea 367.58 ms 437.52 ms 69.94 ms
e2dce0b 308.96 ms 360.10 ms 51.14 ms
83884a0 334.46 ms 400.92 ms 66.46 ms

App size

Revision Plain With Sentry Diff
6405ec5 1.58 MiB 2.12 MiB 552.23 KiB
d5a29b6 1.58 MiB 2.12 MiB 549.37 KiB
f064536 1.58 MiB 2.20 MiB 633.90 KiB
fcec2f2 1.58 MiB 2.12 MiB 551.50 KiB
bbc35bb 1.58 MiB 2.12 MiB 553.01 KiB
694d587 1.58 MiB 2.19 MiB 620.06 KiB
b8bd880 1.58 MiB 2.29 MiB 722.92 KiB
889ecea 1.58 MiB 2.11 MiB 539.75 KiB
e2dce0b 0 B 0 B 0 B
83884a0 1.58 MiB 2.29 MiB 722.97 KiB

@stephanie-anderson stephanie-anderson marked this pull request as ready for review March 27, 2026 14:27
@stephanie-anderson stephanie-anderson enabled auto-merge (squash) March 27, 2026 14:27
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.

@dingsdax dingsdax self-requested a review March 27, 2026 14:34
@stephanie-anderson stephanie-anderson merged commit 6285fcc into main Mar 27, 2026
72 checks passed
@stephanie-anderson stephanie-anderson deleted the chore/add-validate-pr-workflow branch March 27, 2026 14:50
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