search

tree-sitter / node-tree-sitter

Node.js bindings for tree-sitter
https://www.npmjs.com/package/tree-sitter
MIT License
649 stars 114 forks source link

make module context aware #115

Closed guidocalvano closed 2 years ago

guidocalvano commented 2 years ago

Hey, I'm trying to use your awesome bindings in electron, but unfortunately I get complaints that it should not be non-context aware. After digging through stack overflow I found this: https://github.com/electron/electron/issues/18397 where they explain you can make a node library context aware simply by applying the change in this pull request. I tried to build the project myself, but that proved to be quite a lot of work, probably also because I am on an m1 mac. I haven't succeeded yet. Could you have a look at this to see if it works? I could of course not test it given that I cannot yet build the project, even without any changes.

guidocalvano commented 2 years ago

A lot more changes were necessary detailed in an issue. Please have a look and let me know what you think. I ask a few questions too. See https://github.com/tree-sitter/node-tree-sitter/issues/117