search

upsidr / merge-gatekeeper

Get better merge control
MIT License
85 stars 14 forks source link

Update docs to enumerate the specific permissions needed by merge-gatekeeper #63

Closed jdechicchis closed 1 year ago

jdechicchis commented 1 year ago

See issue: https://github.com/upsidr/merge-gatekeeper/issues/62

Based on my usage of merge-gatekeeper, only specifying the following permissions on the GTHUB_TOKEN appear sufficient:

checks: read
statuses: read

Updated the example YAML files to include these permissions. Please let me know if there are any automated tests I should update.

rytswd commented 1 year ago

Thanks for the update, @jdechicchis! We will merge this towards later this week - I don't expect any surprise, but as we use the master branch for our internal Merge Gatekeeper use cases, we would like to ensure we have enough coverage to update Merge Gatekeeper logics if any unwanted behaviour is found 🙏

rytswd commented 1 year ago

Going ahead with the merge for this one -- thanks for your contribution @jdechicchis! 🥰