Skip to content

updated permissions for preview#123

Merged
vdikransamarjian merged 3 commits into
mainfrom
dikran/update-previews
Jun 3, 2026
Merged

updated permissions for preview#123
vdikransamarjian merged 3 commits into
mainfrom
dikran/update-previews

Conversation

@vdikransamarjian

Copy link
Copy Markdown
Contributor

💸 TL;DR

Changed workflow permission from pull-requests: read to pull-requests: write.
Added continue-on-error: true to React to preview request.
Reason: the rocket reaction is just a nice confirmation, so it should not block the preview build if GitHub rejects that API call with a 403.

✅ Checks

  • CI tests (if present) are passing
  • Adheres to code style for repo
  • Contributor License Agreement (CLA) completed if not a Reddit employee

@vdikransamarjian vdikransamarjian requested a review from a team June 3, 2026 23:16
@vdikransamarjian vdikransamarjian merged commit 24fe04b into main Jun 3, 2026
2 checks passed
@vdikransamarjian vdikransamarjian deleted the dikran/update-previews branch June 3, 2026 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants