-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Enable conformance requirement check #76622
Enable conformance requirement check #76622
Conversation
/hold I'm not 100% sure that the place for this is directly in the existing golint script:
Would a separate verify script be better? |
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.
thanks @oomichi 🎉
/lgtm
/kind feature
/priority important-longterm
/assign @timothysc
oh,
+1 |
/sig testing |
/lgtm cancel |
OK, let's add separated script. |
6814793
to
b6437ce
Compare
b6437ce
to
3745ef7
Compare
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.
Please also review and correct the shellcheck errors.
5555b6e
to
b93e169
Compare
@cblecker Thank you so much for your comments, now this becomes green I feel. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: oomichi, spiffxp 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 |
b93e169
to
c76bb08
Compare
/test pull-kubernetes-e2e-gce |
/lgtm |
This seems to work fine as https://prow.k8s.io/view/gcs/kubernetes-jenkins/pr-logs/pull/76622/pull-kubernetes-verify/1118286688490098688 /hold cancel |
What type of PR is this?
/kind bug
What this PR does / why we need it:
This enables conformance test check for the requirements.
Ref: #74432
Local test 1: Success case:
Local test 2: Failure case:
Does this PR introduce a user-facing change?: