-
-
Notifications
You must be signed in to change notification settings - Fork 45
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 linter practice for C# #451
Conversation
Which Linter you want to use ?? |
Hi, firstly I'm really sorry for the long delay. I had a crazy week. @gagandeepp |
|
||
async evaluate(ctx: PracticeContext): Promise<PracticeEvaluationResult> { | ||
if (ctx.packageInspector) { | ||
if (ctx.packageInspector.hasOneOfPackages(['golang.org/x/lint', 'github.com/golangci/golangci-lint'])) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Choose the most used C# linter.
Hi, @gagandeepp , how are you? Is there any progress with this PR? :) |
In C# we used IDE built in code analysis which is available as DLL ,so I am not sure whether this change can be implemented. |
@gagandeepp Look at this article. Some of them should be able to use for this practice. https://medium.com/@michaelparkerdev/linting-c-in-2019-stylecop-sonar-resharper-and-roslyn-73e88af57ebd |
Hi, @gagandeepp , is there any progress? :) |
Sorry I am unable to a make progress on this |
Description
Add linter practice for c# project. Checking for the following packages:
golang.org/x/lint
github.com/golangci/golangci-lint
Motivation and Context
resolves #451
Types of changes
Checklist: