-
Notifications
You must be signed in to change notification settings - Fork 260
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
CI: enable golint #109
CI: enable golint #109
Conversation
e0a77f4
to
210fbe4
Compare
Signed-off-by: Mayank Shah <[email protected]>
210fbe4
to
2738092
Compare
prow.sh: only run alpha tests for latest Kubernetes release
4aff857 Merge pull request kubernetes-csi#109 from pohly/alpha-test-defaults 0427289 Merge pull request kubernetes-csi#110 from pohly/kind-0.9-bazel-build-workaround 9a370ab prow.sh: work around "kind build node-image" failure 522361e prow.sh: only run alpha tests for latest Kubernetes release 22c0395 Merge pull request kubernetes-csi#108 from bnrjee/master b5b447b Add go ldflags using LDFLAGS at the time of compilation 16f4afb Merge pull request kubernetes-csi#107 from pohly/kind-update 7bcee13 prow.sh: update to kind 0.9, support Kubernetes 1.19 df518fb prow.sh: usage of Bazel optional c3afd42 Merge pull request kubernetes-csi#104 from xing-yang/snapshot dde93b2 Update to snapshot-controller v3.0.0 git-subtree-dir: release-tools git-subtree-split: 4aff857
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
could you also add this check to SMB driver? thanks.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andyzhangx, mayankshah1607 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
maybe you could help? @dr0pdb thanks. |
@andyzhangx SMB driver already has this check running - https://github.com/kubernetes-csi/csi-driver-smb/blob/master/.github/workflows/static.yaml |
not needed any more. |
Signed-off-by: Mayank Shah [email protected]
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #90
Special notes for your reviewer:
Does this PR introduce a user-facing change?: