Skip to content

Conversation

mohitsethia
Copy link

Remove deprecated linters that results in warnings.
Closes: #6122

Release Note

Removes deprecated linter `gomnd`

Copy link
Contributor

@squakez squakez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should not have changes in the go.mod.

@mohitsethia
Copy link
Author

Hi @squakez , if we remove gomnd from disabled linter it will be enabled because we have enable-all:true settings. That's why we need to upgrade the linter.

michalvavrik added a commit to michalvavrik/camel-k that referenced this pull request Sep 8, 2025
- closes: apache#6125

Changes:

- drop `gomnd` as it has been replaced by `mnd` (see golangci/golangci-lint#4572) and then removed (see golangci/golangci-lint#5110)
  - addresses warning: `WARN [lintersdb] The linter "gomnd" is deprecated (step 2) and deactivated. It should be removed
   from the list of disabled linters. https://golangci-lint.run/product/roadmap/#linter-deprecation-cycle`
- disable `tenv` as the same functionality is provided by `usetesting` and we enable it with `enable-all: true`
  - addresses warning: `WARN The linter 'tenv' is deprecated (since v1.64.0) due to: Duplicate feature
  another linter. Replaced by usetesting.`
@squakez squakez closed this in #6273 Sep 9, 2025
squakez pushed a commit that referenced this pull request Sep 9, 2025
- closes: #6125

Changes:

- drop `gomnd` as it has been replaced by `mnd` (see golangci/golangci-lint#4572) and then removed (see golangci/golangci-lint#5110)
  - addresses warning: `WARN [lintersdb] The linter "gomnd" is deprecated (step 2) and deactivated. It should be removed
   from the list of disabled linters. https://golangci-lint.run/product/roadmap/#linter-deprecation-cycle`
- disable `tenv` as the same functionality is provided by `usetesting` and we enable it with `enable-all: true`
  - addresses warning: `WARN The linter 'tenv' is deprecated (since v1.64.0) due to: Duplicate feature
  another linter. Replaced by usetesting.`
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.

Deprecated lints
2 participants