add jq to dockerfile

[justinrush]

Is your feature request related to a problem? Please describe.
When running the tool to generate code quality reports its helpful to be able to tee the output to a file and to some human readable output using jq.

Describe the solution you'd like
RUN apk update && apk add --no-cache jq=1.6-r1

Describe alternatives you've considered
maintain a local fork :(

[boring-cyborg]

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

[sayboras]

In my opinion, golangci-lint docker image should be just bare minimum (e.g. base image + golangci-lint binary). So while adding one more tool (like jq) is trivial, it will lead to few implications:

• request to add one more tool
• more responsibilities to maintain versions
• security vulnerabilities

[justinrush]

In my opinion, golangci-lint docker image should be just bare minimum (e.g. base image + golangci-lint binary). So while adding one more tool (like jq) is trivial, it will lead to few implications:

• request to add one more tool
• more responsibilities to maintain versions
• security vulnerabilities

I don't disagree with the sentiment but at the same time using code quality as an output either requires that you run two jobs, one with non-human and output to file, and another to screen... Or create a private image based on the latest, and then keep maintaining that.

If there is some sed/awk magic people are using instead, I'm game, please share. I just don't want to maintain a private image for a pubic tool for something so basic.

[ldez]

I agree with @sayboras.

I guess that you are using the json output format, instead of adding jq to pipe the output, we can just add another output format like "json for human"

May be related to #481

[ldez]

could you provide more information about your use case?

[justinrush]

could you provide more information about your use case?

The use case is this CI task that sticks computer readable information in an artifact and human readable stuff to the screen:

golangci-lint run --out-format code-climate | tee go-code-quality-report.json | jq -r '.[] | "\(.location.path):\(.location.lines.begin) \(.description)"'

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[justinrush]

this is closed by #2386