-
Notifications
You must be signed in to change notification settings - Fork 48
Add build artifacts in repository #362
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
Comments
Is there a particular tool you’re having trouble integrating this with? |
Thank you for replying so quick 🙏 We are developing a static analyzer that integrates with multiple grammars and our build system assume that the We did not had this issue before with A workaround consists in putting the Thank you, |
Got it, thank you for the context. Are you aware that these artifacts are slated to be removed from many other repositories? The tree-sitter 1.0 checklist states:
What is your plan once that 1.0 action item happens? There is actually a workaround already in place, so you don’t have to create your own repo, I just want to make sure you’re well-positioned for the long term before I tell you what it is. |
Thank you for your quick reply. I was not aware of this change for the 1.0 and in this case, it totally changes this ask. We should adapt to it rather than asking you to adapt. Thank you for the pointer, we will then adapt and prepare accordingly! I am closing this issue since this is no longer relevant. |
Tree-sitter grammars generally includes the
parser.c
and thetree_sitter
directory which includes the header files (see example with Python or JavaScript). This helps to integrate with other tools who integrates with tree-sitter. Without these files, it's hard to integrate this grammar with other tree-sitter based tools.There are currently some explanation as to why there is no
parser.c
but downloading theparser.c
and other build artifacts make people lose the versions and other attributes of a build. This makes it pretty hard to include this grammar in project that includes other tree-sitter grammar since it requires a special build while other grammars do not and all their parsers are being versioned.The text was updated successfully, but these errors were encountered: