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

Permissions in examples #210

Merged
merged 2 commits into from
Dec 24, 2021
Merged

Permissions in examples #210

merged 2 commits into from
Dec 24, 2021

Conversation

korthout
Copy link
Owner

GitHub recommends to set the GITHUB_TOKEN permissions to read-only by default. At the same time, our examples did not yet make it clear what permissions are required by this action.

We can simply add the permissions to the examples. This should make it clear what permissions are required, as well as resolve problems for users/repos with read-only defaults.

closes #189

The action requires specific permissions for the GITHUB_TOKEN. Simply
adding them to the examples should be clear enough for most users.
@korthout korthout merged commit f2c8506 into master Dec 24, 2021
@korthout korthout deleted the permissions-in-examples branch December 24, 2021 11:48
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.

Mention the need for permissions
1 participant