Skip to content

maintidx Support #2434

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

Closed
yagipy opened this issue Dec 25, 2021 · 3 comments · Fixed by #2435
Closed

maintidx Support #2434

yagipy opened this issue Dec 25, 2021 · 3 comments · Fixed by #2435
Labels
enhancement New feature or improvement linter: new Support new linter

Comments

@yagipy
Copy link
Contributor

yagipy commented Dec 25, 2021

Your feature request related to a problem? Please describe.

I would like to measure the maintainability index.

Describe the solution you'd like.

Add support for maintidx.

Describe alternatives you've considered.

Run maintidx separately to golangci-lint.

Additional context.

I will be responsible for implementing it myself.

@yagipy yagipy added the enhancement New feature or improvement label Dec 25, 2021
@boring-cyborg
Copy link

boring-cyborg bot commented Dec 25, 2021

Hey, thank you for opening your first Issue ! 🙂 If you would like to contribute we have a guide for contributors.

@bombsimon bombsimon added the linter: new Support new linter label Dec 25, 2021
@butuzov
Copy link
Member

butuzov commented Dec 25, 2021

How this information can be useful? We already have complexity checks in place, which is simpler abstraction of "code complexity" then one from microsoft.

@yagipy
Copy link
Contributor Author

yagipy commented Dec 25, 2021

@butuzov
Thanks for your opinion.

How this information can be useful?

IMO, "code complexity" is difficult to decide based on a single indicator and should be decided by referring to multiple indicators.
I recognize that there is already support for indicators such as cyclomatic complexity, cognitive complexity, etc., and I think no problem in having a maintainability index among them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement linter: new Support new linter
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants