Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add C# support via tree-sitter #43

Merged
merged 13 commits into from
Dec 17, 2024
Merged

Conversation

gnerkus
Copy link
Contributor

@gnerkus gnerkus commented Jun 30, 2024

No description provided.

@gnerkus gnerkus mentioned this pull request Jun 30, 2024
@jayu
Copy link
Contributor

jayu commented Jul 3, 2024

Thank you for the PR, I'm AFK right now but I will review and test it asap.

I'm impressed that you crack on all internals and made it work!

@SnowPowerCore
Copy link

Hello do you have any updates on this, C# is great to be present

@jayu
Copy link
Contributor

jayu commented Dec 11, 2024

Hi, I forgot about it 🤦I Will review and test the pr this week and merge it if everything will be fine

@jayu
Copy link
Contributor

jayu commented Dec 17, 2024

Hi @gnerkus @SnowPowerCore I've checked the PR, run the extension and it works, but I consider C# support at this stage as experimental.
Use cases covered in tests works, but far away from feature parity with other parsers.
I will merge it and release it, but still keep the original issue open as further improvements have to be done.

Anyway, thanks a lot for your input !

@jayu jayu merged commit 11ba763 into codeque-co:master Dec 17, 2024
@SnowPowerCore
Copy link

This is amazing, thank you

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.

3 participants