Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bump golangci-lint in Makefile as well #259

Merged

Conversation

jiriproX
Copy link
Contributor

make didn't work without it. Please keep golangci-lint version consistent in .github/workflows/static-scan.yml and in Makefile

make didn't work without it.

Signed-off-by: Jiri Prokes <jirix.prokes@intel.com>
@jiriproX
Copy link
Contributor Author

@SchSeba please review it and approve it.

@SchSeba
Copy link
Collaborator

SchSeba commented May 16, 2023

LGTM

@coveralls
Copy link

Pull Request Test Coverage Report for Build 4958034830

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 38.156%

Totals Coverage Status
Change from base Build 4784483752: 0.0%
Covered Lines: 422
Relevant Lines: 1106

💛 - Coveralls

@adrianchiris
Copy link
Contributor

LGTM !

to avoid this split brain, it would be nice if the workflow just called make lint :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants