Skip to content
This repository has been archived by the owner on Sep 12, 2023. It is now read-only.

Remove unnecessary comment on gometalinter #50

Merged
merged 1 commit into from
Jan 16, 2020
Merged

Remove unnecessary comment on gometalinter #50

merged 1 commit into from
Jan 16, 2020

Conversation

terrytangyuan
Copy link
Member

@terrytangyuan terrytangyuan commented Jan 15, 2020

The deprecation of gometalinter is official so this comment isn't necessary anymore.

This change is Reviewable

@Jeffwan
Copy link
Member

Jeffwan commented Jan 15, 2020

Happen to see this
/lgtm

@terrytangyuan
Copy link
Member Author

@Jeffwan Thanks!

/assign @gaocegege @richardsliu @johnugeorge

@gaocegege
Copy link
Member

/approve

I think we should move to golangci-lint.

@gaocegege
Copy link
Member

/approve

I think we should move to golangci-lint.

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gaocegege

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit 88a0cd0 into kubeflow:master Jan 16, 2020
@terrytangyuan terrytangyuan deleted the patch-1 branch January 16, 2020 02:09
@terrytangyuan
Copy link
Member Author

We are already using it

georgkaleido pushed a commit to georgkaleido/common that referenced this pull request Jun 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants