-
Couldn't load subscription status.
- Fork 397
add golangci-lint #567
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 golangci-lint #567
Conversation
7398b8d to
605ed5c
Compare
|
@jirfag, it looks like golangci-lint requires |
That sounds plausible, it probably shells out to go tools. |
|
Hello, yes golangci-lint requires Go and also FYI, it's useless to ping Denis: he has not been active on the project for several years. I recommend not pinging directly maintainers especially when you don't know the person, as this can be considered harassment. Remember that you are not alone to do that, and managing a popular OSS is already taking a lot of time. |
|
I do not believe there is any intent to annoy any upstream maintainers, but rather a desire to make them aware of the intent to include their package in the distribution, and to solicit any opinions about that intention. |
|
Don't take me wrong, I just trying to explain a recurring problem, it is neither against a person nor a project nor this PR. There is no major problem here, I'm trying to explain this just to improve the sanity of OSS maintainers. Just a quick demo of the problem with this PR: there 3 messages each of them is pinging Denis. I'm sure there are no bad intentions here but it is bad behavior. Sorry for the off-topic, I hope I answered the original question in my previous message. |
3df0590 to
475b5f2
Compare
475b5f2 to
bacbcc5
Compare
Signed-off-by: Batuhan Apaydın [email protected]
PTAL @dlorenc @jirfag