-
Notifications
You must be signed in to change notification settings - Fork 33
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
Publish on npm #41
Comments
It could, but I'd like to wait until https://github.com/r-lib/tree-sitter-r/tree/next is merged -- I'm working on a fairly substantial overhaul of the grammar. |
@kevinushey in what timeframe do you expect this could be possible? For our purposes, the current version is very usable and having a big change in the grammar in a later release would not be a problem. |
I think we've decided to maintain "main" and "next" separately, just because it would be too costly to try to bring any existing downstream integrations (e.g. in vim / neovim / Emacs) up to speed. Still, we don't have a timeline for publishing to NPM just yet. |
For our case there is no need anymore for this to be published, we've decided to vendor the parsers we're using. |
Makes sense; thanks! |
Now, that the branch in question is long merged, do you intend on doing this now? @kevinushey. |
@DavisVaughan, what do you think? |
Since we are on |
Thank you very much this would help us greatly! |
This grammar is not published on npm, although it is properly working (when #40 is merged).
Can this grammar be published there?
The text was updated successfully, but these errors were encountered: