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

feat: add an option to verify with the JSONSchema #1167

Merged
merged 11 commits into from
Feb 13, 2025
Merged

Conversation

ldez
Copy link
Member

@ldez ldez commented Feb 13, 2025

Fixes #1163

@ldez ldez added the enhancement New feature or request label Feb 13, 2025
@ldez ldez merged commit f7463c5 into golangci:main Feb 13, 2025
43 checks passed
@ldez ldez deleted the feat/verify branch February 13, 2025 22:14
@azazeal
Copy link

azazeal commented Feb 14, 2025

Hi! Thanks for baking this into the action! Much appreciated.

But, I think, there's an omission as the --config flag isn't passed to verify. Relevant excerpt from our logs:

Running [/home/runner/golangci-lint-1.64.5-linux-amd64/golangci-lint config verify] in [/home/runner/work/workflows/workflows/lintapp] ...
Running [/home/runner/golangci-lint-1.64.5-linux-amd64/golangci-lint run --path-prefix=lintapp --config=../.golangci.yml] in [/home/runner/work/workflows/workflows/lintapp] ...
golangci-lint found no issues
Ran golangci-lint in 13865ms

If --config is provided to run, then the same value should be extracted from the run args and passed to verify, but https://github.com/ldez/golangci-lint-action/blob/594fb07c1e4c07cf5aca70f6775681cf2a580d44/src/run.ts#L141 defines the command as a static string (apologies if that's not the case, not very comfortable reading node code).

@ldez
Copy link
Member Author

ldez commented Feb 14, 2025

I created a fix: https://github.com/golangci/golangci-lint-action/releases/tag/v6.4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: verify enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Configuration file verification
2 participants