-
Notifications
You must be signed in to change notification settings - Fork 536
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.
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
update all the static check tools to latest version #1586
Comments
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Apr 27, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Apr 28, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
ceph-csi-bot
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
May 6, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
May 10, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
Rakshith-R
added a commit
to Rakshith-R/ceph-csi
that referenced
this issue
May 24, 2021
This commit addresses ifshort linter issues which checks if short syntax for if-statements is possible. updates: ceph#1586 Signed-off-by: Rakshith R <rar@redhat.com>
Rakshith-R
added a commit
to Rakshith-R/ceph-csi
that referenced
this issue
May 24, 2021
updates: ceph#1586 Signed-off-by: Rakshith R <rar@redhat.com>
Rakshith-R
added a commit
to Rakshith-R/ceph-csi
that referenced
this issue
May 26, 2021
This commit addresses ifshort linter issues which checks if short syntax for if-statements is possible. updates: ceph#1586 Signed-off-by: Rakshith R <rar@redhat.com>
Rakshith-R
added a commit
to Rakshith-R/ceph-csi
that referenced
this issue
May 26, 2021
updates: ceph#1586 Signed-off-by: Rakshith R <rar@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
May 26, 2021
This commit addresses ifshort linter issues which checks if short syntax for if-statements is possible. updates: #1586 Signed-off-by: Rakshith R <rar@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
May 26, 2021
updates: #1586 Signed-off-by: Rakshith R <rar@redhat.com>
Rakshith-R
added a commit
to Rakshith-R/ceph-csi
that referenced
this issue
May 27, 2021
`nlreturn` linter requires a new line before return and branch statements except when the return is alone inside a statement group (such as an if statement) to increase code clarity. This commit addresses such issues. Updates: ceph#1586 Signed-off-by: Rakshith R <rar@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 2, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 4, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 25, 2021
Use %w instead of %v to format errors. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 25, 2021
In few places the error is not nil still it returns nil. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 28, 2021
Use %w instead of %v to format errors. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 28, 2021
Use %w instead of %v to format errors. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 28, 2021
In few places the error is not nil still it returns nil. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jun 28, 2021
In few places the error is not nil still it returns nil. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
Jun 28, 2021
In few places the error is not nil still it returns nil. Updates: #1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
this commit adds `// nolint:cyclop` for the fucntions whose complexity is above 20 Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit disables wrapcheck and gci linters. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit modifies the error of godot, cyclop, paralleltest linter caused due to merged PRs. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit disables the forbidigo linter as this linter forbids the use of fmt.Printf but we need to use it in various part of our codebase. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit disables gomoddirectives linter as it bans use of replace directive. Update: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
this commit adds `// nolint:cyclop` for the fucntions whose complexity is above 20 Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit disables wrapcheck and gci linters. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
yati1998
added a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit modifies the error of godot, cyclop, paralleltest linter caused due to merged PRs. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit disables the forbidigo linter as this linter forbids the use of fmt.Printf but we need to use it in various part of our codebase. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit disables gomoddirectives linter as it bans use of replace directive. Update: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
this commit adds `// nolint:cyclop` for the fucntions whose complexity is above 20 Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit disables wrapcheck and gci linters. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
leseb
pushed a commit
to yati1998/ceph-csi
that referenced
this issue
Jul 22, 2021
This commit modifies the error of godot, cyclop, paralleltest linter caused due to merged PRs. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
Jul 22, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: #1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
Jul 22, 2021
This commit disables the forbidigo linter as this linter forbids the use of fmt.Printf but we need to use it in various part of our codebase. Updates: #1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
Jul 22, 2021
This commit disables gomoddirectives linter as it bans use of replace directive. Update: #1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
Jul 22, 2021
this commit adds `// nolint:cyclop` for the fucntions whose complexity is above 20 Updates: #1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
Jul 22, 2021
This commit disables wrapcheck and gci linters. Updates: #1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
mergify bot
pushed a commit
that referenced
this issue
Jul 22, 2021
This commit modifies the error of godot, cyclop, paralleltest linter caused due to merged PRs. Updates: #1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
nixpanic
pushed a commit
to red-hat-storage/ceph-csi
that referenced
this issue
Aug 9, 2021
nlreturn linter requires a new line before return and branch statements except when the return is alone inside a statement group (such as an if statement) to increase code clarity. This commit addresses such issues. Updates: ceph#1586 Signed-off-by: Rakshith R <rar@redhat.com>
nixpanic
pushed a commit
to red-hat-storage/ceph-csi
that referenced
this issue
Aug 9, 2021
This PR updates the static check tools to the latest version. Further needs to resolve all the errors after updating the version. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
nixpanic
pushed a commit
to red-hat-storage/ceph-csi
that referenced
this issue
Aug 9, 2021
This commit disables the forbidigo linter as this linter forbids the use of fmt.Printf but we need to use it in various part of our codebase. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
nixpanic
pushed a commit
to red-hat-storage/ceph-csi
that referenced
this issue
Aug 9, 2021
This commit disables gomoddirectives linter as it bans use of replace directive. Update: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
nixpanic
pushed a commit
to red-hat-storage/ceph-csi
that referenced
this issue
Aug 9, 2021
this commit adds `// nolint:cyclop` for the fucntions whose complexity is above 20 Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
nixpanic
pushed a commit
to red-hat-storage/ceph-csi
that referenced
this issue
Aug 9, 2021
This commit disables wrapcheck and gci linters. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
nixpanic
pushed a commit
to red-hat-storage/ceph-csi
that referenced
this issue
Aug 9, 2021
This commit modifies the error of godot, cyclop, paralleltest linter caused due to merged PRs. Updates: ceph#1586 Signed-off-by: Yati Padia <ypadia@redhat.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was going through the golangci-lint new release, looks like we have some static issues its good to run all static tools with latest versions
The text was updated successfully, but these errors were encountered: