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.

Sign up for GitHub

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

Jump to bottom

GitHub action to update vendor files #61

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

GitHub action to update vendor files #61

wants to merge 7 commits into from

Conversation

smacker
Copy link
Owner

@smacker smacker commented Feb 9, 2022 •
edited
Loading

The same as #59 with the following changes:

  • remove commit that changes lua grammar
  • remove empty workflow
  • run tests before committing changes

@smacker
Copy link
Owner Author

smacker commented Feb 9, 2022

This job will only update grammars that point to master.
We probably want to modify it to automatically bump grammar pinned versions too.

Example run of the workflow: https://github.com/smacker/go-tree-sitter/runs/5120032143?check_suite_focus=true

@smacker smacker mentioned this pull request Feb 9, 2022
Closed
@NickyBoy89
Copy link

NickyBoy89 commented Feb 9, 2022 •
edited
Loading

Looks good! Thank you for all the help with this @smacker

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@smacker @NickyBoy89