Skip to content

Conversation

@weiji14
Copy link
Member

@weiji14 weiji14 commented Sep 24, 2025

Was getting Warning: Failed to set reaction on comment ID XXX and Adding reaction 'hooray' to comment id 'XXX' failed due to missing write permissions to pull-requests. Noticed this at #602 (comment), where earlier comments had emoji reactions, but latter ones didn't.

Changes to set permissions: pull-requests: write where needed, and also changed from using PANGEOBOT_TOKEN to the default GITHUB_TOKEN in one instance.

References:

Was getting `Warning: Failed to set reaction on comment ID XXX` and `Adding reaction 'hooray' to comment id 'XXX' failed` due to missing write permissions to pull-requests.
@weiji14 weiji14 self-assigned this Sep 24, 2025
@weiji14 weiji14 merged commit d056b05 into master Sep 25, 2025
5 of 6 checks passed
@weiji14 weiji14 deleted the chatops-permissions branch September 25, 2025 00:18

permissions:
issues: write
contents: read
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, this might actually need to be contents: write according to peter-evans/repository-dispatch#196 (comment). Otherwise we end up with Error: Resource not accessible by integration (see https://github.com/pangeo-data/pangeo-docker-images/actions/runs/17993095086/job/51187138177 triggered by #623 (comment)).

weiji14 added a commit that referenced this pull request Sep 25, 2025
Fixes `Error: Resource not accessible by integration`, patches #625.
weiji14 added a commit that referenced this pull request Sep 25, 2025
…626)

Fixes `Error: Resource not accessible by integration`, patches #625.
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