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

integration with golangci-lint #2

Open
IAD opened this issue Dec 16, 2020 · 2 comments
Open

integration with golangci-lint #2

IAD opened this issue Dec 16, 2020 · 2 comments

Comments

@IAD
Copy link

IAD commented Dec 16, 2020

Hi Ole, do we plan to include the tool in the https://github.com/golangci/golangci-lint ?

@ole108
Copy link
Member

ole108 commented Dec 16, 2020

Hi Igor,
that is a great idea.
I plan some documentation improvements and added functionality for generated documentation (dependency matrix) until end of January.
The command line arguments and configuration will stay the same (except for added flags for generating documentation).
With that addition I will move the version to 1.0.

How can I help with the integration?

Cheers,
Ole

@ole108
Copy link
Member

ole108 commented Dec 16, 2020

I just found on the integration page:

You need to implement a new linter using go/analysis API. We don't accept not go/analysis linters.

The spaghetti-cutter doesn't use go/analysis so far. I don't know if it would be possible to change.
I will try to find out. Help for that would be appreciated.
The spaghetti-cutter needs access to all packages (full names) of a project and their imports.

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

2 participants