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

Add C# language support #276

Merged
merged 2 commits into from
Apr 17, 2020
Merged

Add C# language support #276

merged 2 commits into from
Apr 17, 2020

Conversation

damieng
Copy link
Contributor

@damieng damieng commented Apr 8, 2020

Add's C# support here in order to get closer to C# semantic code support on GitHub.com

Not sure how to test this locally but the CI/GH Actions are passing including running the corpus tests.

Let me know if anything else needs to happen either on this repo, tree-sitter-c-sharp or elsewhere!

Fixes #185

@damieng damieng mentioned this pull request Apr 9, 2020
@damieng damieng marked this pull request as ready for review April 9, 2020 01:05
@damieng damieng changed the title Add tree-sitter-c-sharp support Add C# language support Apr 9, 2020
@damieng damieng requested a review from patrickt April 9, 2020 02:31
Copy link
Contributor

@patrickt patrickt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me pending @aymannadeem’s sign-off.

Copy link
Contributor

@aymannadeem aymannadeem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great!

@damieng
Copy link
Contributor Author

damieng commented Apr 17, 2020

Okay to merge?

@aymannadeem
Copy link
Contributor

Fine by me

@damieng damieng merged commit 3c69c9c into tree-sitter:master Apr 17, 2020
@damieng damieng deleted the add-tree-sitter-c-sharp branch April 17, 2020 18:37
@damieng
Copy link
Contributor Author

damieng commented Apr 17, 2020

Should I create a PR for a version bump and change-log ?

@patrickt
Copy link
Contributor

Since this is the first version, we shouldn’t need any changeling updates: we should just do the rest of the steps in the release process and pop it up on Hackage.

@damieng
Copy link
Contributor Author

damieng commented May 6, 2020

Do you need me to do those steps or is the "we" github?

@patrickt
Copy link
Contributor

patrickt commented May 7, 2020

@damieng I think for now @tree-sitter/semantic-code should own the tree-sitter packages, though we’ll be happy to add you alongside, since you did the work!

For now, you can depend on the new package directly as a pinned git version in the cabal.project file in Semantic. That should unblock you on subsequent steps (creation of semantic-c-sharp, etc.).

@jimhester jimhester mentioned this pull request Dec 7, 2020
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.

Add incomplete tree-sitter-c-sharp package
3 participants