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.

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

Upgrade deprecated linters #3012

Closed
squakez opened this issue Feb 15, 2022 · 3 comments
Closed

Upgrade deprecated linters #3012

squakez opened this issue Feb 15, 2022 · 3 comments
Assignees
@Gre8t
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 Feb 15, 2022

Lint execution is producing these warnings:

  level=warning msg="[runner] The linter 'interfacer' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner. "
  level=warning msg="[runner] The linter 'golint' is deprecated (since v1.41.0) due to: The repository of the linter has been archived by the owner.  Replaced by revive."
  level=warning msg="[runner] The linter 'scopelint' is deprecated (since v1.39.0) due to: The repository of the linter has been deprecated by the owner.  Replaced by exportloopref."

We should fix them.
@squakez squakez added the area/continuous integration Related to CI and automated testing label Feb 15, 2022
@squakez squakez added the good first issue Does not require full understanding of the codebase label Mar 21, 2022
@Gre8t
Copy link
Contributor

Gre8t commented Mar 28, 2022

Hello, how do I reproduce these warnings and could this be assigned to me.

@squakez
Copy link
Contributor Author

squakez commented Mar 28, 2022

You can run make lint to test locally.

This was referenced Mar 28, 2022
Closed
Merged
@Gre8t
Copy link
Contributor

Gre8t commented Apr 9, 2022 •
edited
Loading

I think we can close this issue

@squakez squakez closed this as completed May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Gre8t Gre8t

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

No branches or pull requests
2 participants
@Gre8t @squakez