Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: give depandabot permissions to update PR status #1770

Merged
merged 2 commits into from
May 23, 2023
Merged

Conversation

toidiu
Copy link
Contributor

@toidiu toidiu commented May 19, 2023

Description of changes:

Based on docs and a test dependabot PR, the ouzi-dev/commit-status-updater task is failing.

ouzi requires the statuses: write permissions to work, which I add in this PR.

Testing

Testing this would require one of the two methods:

  • merge this to see if it works (recommended)
  • try in a separate repo

Since the statuses: write is a safe permission and I am pretty certain this should help resolve the issue, I think we should just merge this and test it out. I also compared the failing CI's permissions with a successful CI run and can see that the Statuses: read, further confirming my theory.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@toidiu toidiu merged commit 0e723be into main May 23, 2023
@toidiu toidiu deleted the ak-dependabotperms branch May 23, 2023 21:37
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