Skip to content

Migrate to tree-sitter-languages-freed-wu-pr: expand language support#10

Closed
FeepingCreature wants to merge 2 commits into
Aider-AI:mainfrom
FeepingCreature:main
Closed

Migrate to tree-sitter-languages-freed-wu-pr: expand language support#10
FeepingCreature wants to merge 2 commits into
Aider-AI:mainfrom
FeepingCreature:main

Conversation

@FeepingCreature

@FeepingCreature FeepingCreature commented Oct 28, 2024

Copy link
Copy Markdown

I want it for D, so I added a D entry. I don't know what the extensions of the other new languages are.

I didn't see #8 until just now so there's some amount of redundant effort here. Oops :) tree-sitter-languages-freed-wu-pr is a microfork I made based on grantjenks/py-tree-sitter-languages#53 . It does not change the API or build behavior, it just adds new languages (at least on Linux/MacOS, I couldn't get the Windows linker to work with the full list). This change is smaller, but #8 has more hope of support. So this is intended as a band-aid.

@FeepingCreature

Copy link
Copy Markdown
Author

Ping?

@paul-gauthier

Copy link
Copy Markdown
Collaborator

I appreciate the effort here! I am nervous to move from one unsupported package to another unsupported package.

I am hopeful about this package from the Textual folks though:

https://github.com/Textualize/py-tree-sitter-languages

@FeepingCreature

FeepingCreature commented Nov 30, 2024

Copy link
Copy Markdown
Author

I mean that's very fair, I'm not proposing it as a long-term solution. In my opinion, unless I've majorly messed up the release process (which is very possible!), it should be better than what you have right now.

tree-sitter wants now to be used in a format where you have small binaries for every language, and that's definitely the way to go moving forward, but I don't think it's ready for deployment everywhere yet. And I mean, personally, my workplace is a D company, and I kinda would like to avoid telling everyone to install a branch... :P

(Though it also wouldn't be a huge hindrance, we can use git/pip.)

@FeepingCreature

FeepingCreature commented Dec 22, 2024

Copy link
Copy Markdown
Author

Apparently https://pypi.org/project/tree-sitter-language-pack/ is a thing now too. Bit less ad-hoc :) If that'd be acceptable, I'll give that a poke after the holidays.

@FeepingCreature

Copy link
Copy Markdown
Author

Hooray! Thanks for adding TSLP! Gonna close this then :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants