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

Set up permissions for GitHub Workflows #217

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joycebrum
Copy link

Closes #216

Changes

  • Set main.yml top level permission to contents: read

Test

I've tested and the workflow ran with success
https://github.com/joycebrum/antlr3/actions/runs/4246430550

Signed-off-by: Joyce <joycebrum@google.com>
@joycebrum
Copy link
Author

Hi!

I'm writing to follow up on your review in this pull request. I noticed that you haven't left any feedback yet.

I'm happy to answer any questions you have about my changes. I'm also available to chat if you'd like to discuss my changes in more detail.

If you have any concerns about my changes, please let me know so that I can address them.

If the PR goes idle for the next 2 months, I'll close it. If you'd rather keep it open as "not planned", just let me know and I'll keep it open even if it gets idle again.

Thanks!

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.

Set up permissions for GitHub Workflows
1 participant