-
-
Notifications
You must be signed in to change notification settings - Fork 152
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
GitHub Annotations suppresses normal output #362
Comments
Hey, thank you for opening your first Issue ! 🙂 If you would like to contribute we have a guide for contributors. |
I think this is a limitation of how the underlying golangci-lint output options work |
I solved this issue by running the linter two times:
|
@ldez this is not a bug, it is a limitation of the output format, probably we can allow to define multiple output formats |
My first thought had been multiple output formats, but on some reflection, just including the file/line number in the annotation maybe cleaner. Or both. I'm happy to try to put up a PR, though I don't know which of the 3 approaches is likely to be accepted. |
I would really like the file + line number context in both areas as well. |
In golangci-lint it's now possible to have multiple output formats. |
Welcome
Description of the problem
I really love that the github action supports annotations. But when using it, there is minimal other output. Specifically, while the message is printed, it lacks the file/line number. (And at least sometimes, Github itself doesn't display anything at all)
Everything is in the annotations, but those don't display properly in a couple of cases:
I'm not sure if this is a bug or a feature request...
I'd suggest either a normal print along side the github actions print, or allowing multiple out-formats to be specified.
Version of golangci-lint
Configuration file
N/A
Go environment
GitHub Actions
Verbose output of running
No Output:
Minimal Output:
None of those include file/line number
Code example or link to a public repository
https://github.com/kolide/launcher/runs/4039960409?check_suite_focus=true
The text was updated successfully, but these errors were encountered: