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
Move proto format ci check to proto job #4611
Conversation
0b5a408
to
2e6d7b9
Compare
Build succeeded.
|
.github/workflows/ci.yml
Outdated
- name: Check fmt | ||
working-directory: src/github.com/containerd/containerd | ||
run: | | ||
export PATH=$PATH:$(go env GOPATH)/bin |
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.
We already update PATH
in Set env
step above:
https://github.com/containerd/containerd/pull/4611/files#diff-e9f950f17198d3d5e3122a44230a09b9R137
I think this line should not be needed.
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.
Removed, let's see what ci says :)
Signed-off-by: Derek McGowan <derek@mcg.dev>
2e6d7b9
to
8b20ca9
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.
LGTM
Build succeeded.
|
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
Follow up to lint change