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

fix govet issues reported in latest golangci-lint versions #38

Merged
merged 1 commit into from
Aug 26, 2024
Merged

fix govet issues reported in latest golangci-lint versions #38

merged 1 commit into from
Aug 26, 2024

Conversation

tariq1890
Copy link
Contributor

Fixes the following issue

run golangci-lint
  Running [/home/runner/golangci-lint-1.60.3-linux-amd64/golangci-lint run  -v --timeout 5m] in [/home/runner/work/k8s-driver-manager/k8s-driver-manager] ...
  cmd/nvdrain/main.go:140:14: printf: non-constant format string in call to (*github.com/sirupsen/logrus.Logger).Fatalf (govet)
  		log.Fatalf(err.Error())
  		           ^

@tariq1890
fix govet issues reported in latest golangci-lint versions
6d7daed 
Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
@tariq1890 tariq1890 merged commit 72194ac into master Aug 26, 2024
5 checks passed
@tariq1890 tariq1890 deleted the fix-lint branch August 26, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cdesiniotis cdesiniotis cdesiniotis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tariq1890 @cdesiniotis