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

feat: complete the grammar and improve queries #2

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

Conversation

amaanq
Copy link

@amaanq amaanq commented Jan 31, 2023

cc: @liketurbo as I'm sure he'd like to see this

This PR adds a few missing parts of the grammar, such as multiline asm strings, const vars, include and pragma directives, and try/catch clauses.

It also adds a CI action to ensure all future commits/PRs are good, a linter for good code formatting+styling, and a few tidy-ups to the manifests, .git files, and README.

I am submitting this so that when this is merged tree-sitter-func can then be added to nvim-treesitter as a supported language.

Thanks!
-Amaan

@amaanq
Copy link
Author

amaanq commented Feb 2, 2023

While it's only been a day, I am not very hopeful this will be merged anytime soon as @akifoq's activity seems to be next to nothing, so I will add my fork to nvim-treesitter until then. I would love to give any nvim-treesitter users the ability to install func easily, so I hope to hear from a maintainer soon! (Also open to being added as a contributor)

Thanks,

@amaanq amaanq force-pushed the master branch 2 times, most recently from ea161a0 to 88ee7d0 Compare February 2, 2023 13:25
@amaanq amaanq force-pushed the master branch 2 times, most recently from e6404d3 to 9200ee8 Compare May 13, 2023 05:21
github-actions bot and others added 3 commits May 13, 2023 01:22
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants