-
Notifications
You must be signed in to change notification settings - Fork 22
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
Permissions are not documented #147
Comments
The action tries to create the PR label if it doesn't exist already. PRs don't have access to do do that. You can work around this error by pre-creating the label manually. |
I created I guess it fails later while trying to add a label and comment to PR. That functionality is also not available to PRs from forks. But at least it produces a file list, even if the last error message is confusing. |
Maybe |
I fixed the error by adding the jobs:
check:
name: Check
runs-on: ubuntu-latest
permissions:
# required for ppremk/lfs-warning
pull-requests: write
steps:
... |
Is your feature request related to a problem? Please describe.
I tried to use lfs-warnings for
pull_request
events from forks of a public repo. I got "Resource not accessible by integration" error.PR in question: FerretDB/FerretDB#1371
Describe the solution you'd like
lfs-warnings should document what permissions it needs.
The text was updated successfully, but these errors were encountered: