Skip to content

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

Merged
JPeer264 merged 1 commit into
developfrom
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 into
developfrom
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>
@JPeer264JPeer264 self-assigned this Apr 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser26.16 kB--
@sentry/browser - with treeshaking flags24.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 flags72.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/react27.9 kB--
@sentry/react (incl. Tracing)46.35 kB--
@sentry/vue31.03 kB--
@sentry/vue (incl. Tracing)45.95 kB--
@sentry/svelte26.18 kB--
CDN Bundle28.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 - uncompressed84.55 kB--
CDN Bundle (incl. Tracing) - uncompressed139.68 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed88.75 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed143.14 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed212.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed257.49 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed260.93 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed271.18 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed274.62 kB--
@sentry/nextjs (client)48.84 kB--
@sentry/sveltekit (client)44.56 kB--
@sentry/node-core58.96 kB+0.02%+8 B 🔺
@sentry/node170.25 kB+0.01%+11 B 🔺
@sentry/node - without tracing96.82 kB+0.01%+8 B 🔺
@sentry/aws-serverless113.69 kB+0.03%+33 B 🔺
@sentry/cloudflare (withSentry) - minified164.13 kB--
@sentry/cloudflare (withSentry)415.2 kB--

View base workflow run

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

@JPeer264@mydea