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

Make JSON available via Outputs #964

Closed
wants to merge 4 commits into from
Closed

Conversation

tgorham4
Copy link

@tgorham4 tgorham4 commented Feb 5, 2024

It can be useful to pass the machine-readable results of a linting run to later CI steps to power customizations like notifications or PR comments

This change requests JSON output from golangci-lint, grabs the results, and makes them available in the action's outputs under the key JSON

@ldez ldez added enhancement New feature or request blocked Need's direct action from maintainer labels Feb 5, 2024
@tgorham4 tgorham4 changed the title Make JSON available via Github Action Output Make JSON available via Outputs Feb 5, 2024
@ldez ldez added declined and removed blocked Need's direct action from maintainer labels Apr 29, 2024
@ldez
Copy link
Member

ldez commented Apr 29, 2024

Hello,

You can do the same thing by using args: --out-format=json:afile.json.
I think it isn't worth it to always generate JSON reports only for a niche and with other existing solutions.

@ldez ldez closed this Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
declined enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants