tree-sitter-ocaml icon indicating copy to clipboard operation
tree-sitter-ocaml copied to clipboard

Bump to version 20 of tree-sitter

Open VixieTSQ opened this issue 3 years ago • 5 comments

I tried to do this myself but this repo is different most tree-sitter repos as it really has 2 parsers. I couldn't figure out how exactly this should work in the bindings.

VixieTSQ avatar Jun 18 '22 04:06 VixieTSQ

I updated tree-sitter and did a rebuild.

@maxbrunsfeld Can you release a new version on NPM and crates.io?

314eter avatar Jun 18 '22 21:06 314eter

I updated tree-sitter and did a rebuild.

@314eter The rust binding still depends on tree-sitter 0.19 in cargo.toml.

VixieTSQ avatar Jun 19 '22 01:06 VixieTSQ

Fixed in #58.

314eter avatar Jun 19 '22 19:06 314eter

This still needs a release.

torhovland avatar Sep 08 '22 07:09 torhovland

@maxbrunsfeld Can you create a new release?

314eter avatar Sep 14 '22 00:09 314eter

I've created a new release: https://github.com/tree-sitter/tree-sitter-ocaml/releases/tag/v0.20.0

aryx avatar Dec 12 '22 15:12 aryx

Can we close the issue?

aryx avatar Dec 12 '22 15:12 aryx

@aryx I created a new release with the latest changes. Do you have access to the tree-sitter organization on crates.io and NPM to publish the new version there? I can only create releases on Github.

314eter avatar Dec 14 '22 18:12 314eter

I think we could add an action like they did for tree-sitter-java. See this thread: https://github.com/tree-sitter/tree-sitter-java/pull/98

aryx avatar Dec 15 '22 09:12 aryx

That looks like a good solution for crates.io. It would be nice to have something similar for NPM. I see https://www.npmjs.com/package/tree-sitter-java is still on 0.19.1 too.

314eter avatar Dec 15 '22 09:12 314eter

Should this issue still be open? It looks like the issue is, is that the new changes weren't pushed to crates.io? Am I correct in that?

VixieTSQ avatar Dec 19 '22 06:12 VixieTSQ

I'll close this one and create a new issue to implement the publish action.

314eter avatar Dec 19 '22 13:12 314eter