Skip to content

Deprecated lints #6122

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.

Sign up for GitHub

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

Jump to bottom
Open
squakez opened this issue Apr 2, 2025 · 4 comments · May be fixed by #6125
Open

Deprecated lints #6122

squakez opened this issue Apr 2, 2025 · 4 comments · May be fixed by #6125
Labels
area/continuous integration Related to CI and automated testing good first issue Does not require full understanding of the codebase

Comments

@squakez
Copy link
Contributor

squakez commented Apr 2, 2025 •
edited
Loading 

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 
WARN The linter 'tenv' is deprecated (since v1.64.0) due to: Duplicate feature another linter. Replaced by usetesting.
@squakez squakez added kind/feature New feature or request good first issue Does not require full understanding of the codebase area/continuous integration Related to CI and automated testing and removed kind/feature New feature or request labels Apr 2, 2025
@mohitsethia
Copy link

Hey @squakez , I am new to this repo and want to contribute. We only need to remove the gomnd & tenv linters, right?

@squakez
Copy link
Contributor Author

squakez commented Apr 3, 2025

Hello, thanks for the interest. Yes, that's what is required by this issue. Feel free to contribute.

@mohitsethia
Copy link

Thanks, then I will make the changes and share the PR.

@mohitsethia mohitsethia linked a pull request Apr 3, 2025 that will close this issue
Open
@mohitsethia
Copy link

I only found gomnd linter, but not tenv, probably that it is in enabled-all. So I upgraded the linter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/continuous integration Related to CI and automated testing good first issue Does not require full understanding of the codebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove deprecated linters mohitsethia/camel-k
2 participants
@squakez @mohitsethia