Skip to content

chore(ci): Use pull-requests: write for PR review reminder workflow#20597

Merged
JPeer264 merged 1 commit intodevelopfrom
jp/fix-review-reminder-permissions
Apr 29, 2026
Merged

chore(ci): Use pull-requests: write for PR review reminder workflow#20597
JPeer264 merged 1 commit intodevelopfrom
jp/fix-review-reminder-permissions

Conversation

@JPeer264
Copy link
Copy Markdown
Member

The schedule-triggered workflow was failing with 403 when trying to create comments on PRs. GitHub requires pull-requests: write (not issues: write) to create comments on pull requests via the Issues API when the workflow runs on schedule.

ref: https://github.com/getsentry/sentry-javascript/actions/workflows/pr-review-reminder.yml

The schedule-triggered workflow was failing with 403 when trying to
create comments on PRs. GitHub requires `pull-requests: write` (not
`issues: write`) to create comments on pull requests via the Issues API
when the workflow runs on schedule.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@JPeer264 JPeer264 requested review from logaretm and nicohrubec April 29, 2026 11:36
@JPeer264 JPeer264 self-assigned this Apr 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 26.16 kB - -
@sentry/browser - with treeshaking flags 24.63 kB - -
@sentry/browser (incl. Tracing) 44.11 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 46.16 kB - -
@sentry/browser (incl. Tracing, Profiling) 49.06 kB - -
@sentry/browser (incl. Tracing, Replay) 83.46 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 72.94 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 88.14 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 100.79 kB - -
@sentry/browser (incl. Feedback) 43.4 kB - -
@sentry/browser (incl. sendFeedback) 30.96 kB - -
@sentry/browser (incl. FeedbackAsync) 36.14 kB - -
@sentry/browser (incl. Metrics) 27.44 kB - -
@sentry/browser (incl. Logs) 27.59 kB - -
@sentry/browser (incl. Metrics & Logs) 28.28 kB - -
@sentry/react 27.9 kB - -
@sentry/react (incl. Tracing) 46.35 kB - -
@sentry/vue 31.03 kB - -
@sentry/vue (incl. Tracing) 45.95 kB - -
@sentry/svelte 26.18 kB - -
CDN Bundle 28.84 kB - -
CDN Bundle (incl. Tracing) 46.71 kB - -
CDN Bundle (incl. Logs, Metrics) 30.25 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 47.8 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 69.34 kB - -
CDN Bundle (incl. Tracing, Replay) 83.88 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 84.94 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 89.69 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 90.78 kB - -
CDN Bundle - uncompressed 84.55 kB - -
CDN Bundle (incl. Tracing) - uncompressed 139.68 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 88.75 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 143.14 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 212.71 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 257.49 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 260.93 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 271.18 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 274.62 kB - -
@sentry/nextjs (client) 48.84 kB - -
@sentry/sveltekit (client) 44.56 kB - -
@sentry/node-core 58.96 kB +0.02% +8 B 🔺
@sentry/node 170.25 kB +0.01% +11 B 🔺
@sentry/node - without tracing 96.82 kB +0.01% +8 B 🔺
@sentry/aws-serverless 113.69 kB +0.03% +33 B 🔺
@sentry/cloudflare (withSentry) - minified 164.13 kB - -
@sentry/cloudflare (withSentry) 415.2 kB - -

View base workflow run

@JPeer264 JPeer264 merged commit 0bd937b into develop Apr 29, 2026
75 checks passed
@JPeer264 JPeer264 deleted the jp/fix-review-reminder-permissions branch April 29, 2026 12:05
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