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

New language support : C# #757

Open
ldubrois opened this issue Nov 7, 2018 · 8 comments
Open

New language support : C# #757

ldubrois opened this issue Nov 7, 2018 · 8 comments

Comments

@ldubrois
Copy link

ldubrois commented Nov 7, 2018

I've started using Danger on our Gitlab repository but in my team we are mainly C# developers and ruby is not well known by us.

I would like to provide a.net core implementation of Danger. Does it make sense to you?

What will be for you the first step to begins smoothly?

Thanks.

@orta
Copy link
Member

orta commented Nov 7, 2018

It makes perfect sense - assuming you're building it in C#, I've created a repo: https://github.com/danger/csharp

My C# skills are pretty rusty (been about 10 years) so I can only be so useful. I have a document specifically on how to build a language runner: https://danger.systems/js/usage/danger-process.html

Roughly: danger-js will send a JSON payload of the DSL, you convert that DSL into types in C# ( this might help ) and find a way to execute/eval code with those set up at runtime. You then need to console.log out a file path for danger-js to pick up the results json from once your app's process has finished

@orta
Copy link
Member

orta commented Nov 7, 2018

I can help out with CI etc, and if it's there's a way for me to run on mono / macOS I can try help out too

@orta
Copy link
Member

orta commented Nov 7, 2018

And finally, I think this is generally a useful checklist of things you can aim for to make a complete implementation: danger/swift#67

@orta
Copy link
Member

orta commented Nov 7, 2018

Ah, yeah, that relies on GitLab support existing in danger-js, which doesn't exist yet: #396

But that's still way less effort than building all of the communications/reporting/CI/commands

@ldubrois
Copy link
Author

Sorry for my late answer.

I've take a look on the different link you proposed, thanks for it.

I will have to check if C# parser and on the fly compiler are available on .net core (which is officlial supported on linux, mac, and of course windows), otherwise I will have to use the full .net framework which is only available on Windows (and check with Mono).

@orta
Copy link
Member

orta commented Aug 15, 2019

Did this get anywhere? Gitlab has been added since

@acherkashin
Copy link
Contributor

acherkashin commented Jan 26, 2022

@orta any updates on C# implementation? It seems, C# repository has been removed.

@orta
Copy link
Member

orta commented Jan 26, 2022 via email

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

No branches or pull requests

3 participants