Skip to content

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

Merged
jayu merged 13 commits into
codeque-co:masterfrom
gnerkus:csharp-support
Dec 17, 2024
Merged

jayu merged 13 commits into
codeque-co:masterfrom
gnerkus:csharp-support

Conversation

@gnerkus

@gnerkus gnerkus commented Jun 30, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@gnerkus gnerkus mentioned this pull request Jun 30, 2024
@jayu

jayu commented Jul 3, 2024

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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

@jayu

jayu commented Dec 11, 2024

Copy link
Copy Markdown
Contributor

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

@jayu

jayu commented Dec 17, 2024

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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