Skip to content

Conversation

@dveeden
Copy link
Collaborator

@dveeden dveeden commented Feb 16, 2025

allow-leading-space: false is not valid config anymore

$ golangci-lint config verify
jsonschema: "linters-settings.nolintlint" does not validate with "/properties/linters-settings/properties/nolintlint/additionalProperties": additionalProperties 'allow-leading-space' not allowed
Failed executing command with error: the configuration contains invalid elements

Also allow annotations:
https://github.com/golangci/golangci-lint-action/tree/v6/?tab=readme-ov-file#annotations

Needed for annotations to work:

  • Permission
  • setup-go (for the problem matcher)

Also set the Go version to 'stable' instead of a specific version and add 1.24 to the list.

`allow-leading-space: false` is not valid config anymore

```
$ golangci-lint config verify
jsonschema: "linters-settings.nolintlint" does not validate with "/properties/linters-settings/properties/nolintlint/additionalProperties": additionalProperties 'allow-leading-space' not allowed
Failed executing command with error: the configuration contains invalid elements
```
@dveeden dveeden marked this pull request as draft February 16, 2025 11:08
@dveeden dveeden force-pushed the ci_nolintlint_leading_space branch from 32588ad to 19f0abb Compare February 16, 2025 11:30
@dveeden dveeden marked this pull request as ready for review February 16, 2025 11:30
@dveeden dveeden marked this pull request as draft February 16, 2025 11:45
@dveeden dveeden marked this pull request as draft February 16, 2025 11:45
@dveeden dveeden force-pushed the ci_nolintlint_leading_space branch from 63c341a to 0ab5c04 Compare February 16, 2025 11:48
@dveeden dveeden marked this pull request as ready for review February 16, 2025 11:49
@lance6716 lance6716 merged commit cf91430 into go-mysql-org:master Feb 17, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants