search

CyberShadow / tree-sitter-d

17 stars 6 forks source link

Support integration in text editor #8

Closed ErdoganSeref closed 1 year ago

ErdoganSeref commented 1 year ago

The change resolves #6. Also the change helps resolve #4072 in helix.

To ease usage of the grammar via helix we stop ignoring generated parser.h and parser.c.

CyberShadow commented 1 year ago

Thanks! I need to think about how to fit this with the project's build flow.

Does Helix not need queries (for syntax highlighting etc.)? Those would be another big missing piece for general-purpose usage.

WebFreak001 commented 1 year ago

I don't think it's a great idea to put binaries / artifacts (parser.c) into the source repository, it would probably make more sense to put those into a release archive, e.g. from github actions. Does helix support loading the parser from a release zip file?

ErdoganSeref commented 1 year ago

Does Helix not need queries (for syntax highlighting etc.)? Those would be another big missing piece for general-purpose usage.

Yes helix needs queries.

I don't think it's a great idea to put binaries / artifacts (parser.c) into the source repository, it would probably make more sense to put those into a release archive, e.g. from github actions. Does helix support loading the parser from a release zip file?

I don't think so.