search

r-lib / tree-sitter-r

MIT License
111 stars 32 forks source link

Publish on npm #41

Open rien opened 1 year ago

rien commented 1 year ago

This grammar is not published on npm, although it is properly working (when #40 is merged).

Can this grammar be published there?

kevinushey commented 1 year ago

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.

rien commented 11 months ago

@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.

kevinushey commented 10 months ago

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.

rien commented 10 months ago

For our case there is no need anymore for this to be published, we've decided to vendor the parsers we're using.

kevinushey commented 10 months ago

Makes sense; thanks!