Skip to content

Fix: make reaction step non-blocking, add pull-requests permission#1813

Merged
jaxdesmarais merged 1 commit into
mainfrom
fix-inner-source-labels
Jun 3, 2026
Merged

Fix: make reaction step non-blocking, add pull-requests permission#1813
jaxdesmarais merged 1 commit into
mainfrom
fix-inner-source-labels

Conversation

@jaxdesmarais

Copy link
Copy Markdown
Contributor

Summary of changes

  • The reaction step was failing with "Resource not accessible by integration", blocking the label step from running. Adding continue-on-error to the acknowledge steps so label actions proceed even if reactions are restricted. Also adds pull-requests: write permission which may be required for PR label operations.

AI Usage

Which AI Agent Was Used?

  • Copilot
  • Claude
  • Other (Type Name Here)

How was AI used?

Estimated AI Code Contribution

  • less than 30%
  • 30 - 60%
  • 60 - 100%

Checklist

  • [ ] Added a changelog entry
  • [ ] Tested and confirmed payment flows affected by this change are functioning as expected

Authors

The reaction step was failing with "Resource not accessible by
integration", blocking the label step from running. Adding
continue-on-error to the acknowledge steps so label actions proceed
even if reactions are restricted. Also adds pull-requests: write
permission which may be required for PR label operations.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@jaxdesmarais jaxdesmarais requested a review from a team June 1, 2026 20:40
@jaxdesmarais jaxdesmarais requested a review from a team as a code owner June 1, 2026 20:40
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities found.

Scanned Files

None

@jaxdesmarais jaxdesmarais merged commit 31cb892 into main Jun 3, 2026
36 of 38 checks passed
@jaxdesmarais jaxdesmarais deleted the fix-inner-source-labels branch June 3, 2026 18:19
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.

3 participants