-
Notifications
You must be signed in to change notification settings - Fork 52
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
lstat /sh: no such file or directory #297
Comments
@shanduur I didn't expect that someone else wanted to build the protolint by themselves.
Let me know what you think. |
No, no. I wanted to run the Protolint in the CI. Unfortunately, when using the docker image that you provided, the CI fails with the message I provided above:
When I use |
Thank you for telling me. I'm not familiar with GitLab CI. |
You fan create empty repository and add two files:
protolint:
image: docker.io/yoheimuta/protolint:latest
script:
- protolint lint .
only:
- main
- merge_requests
syntax = "proto3";
package component_api;
message Component {
string name = 1;
uint32 status = 2;
}
message ComponentResponse {
Component component = 1;
string error = 3;
} This would run on every commit to main branch and on every commit in open merge request (pull request). |
@shanduur I reproduced this issue. [1] The immediate fix is resetting the entrypoint in protolint:
image:
name: docker.io/yoheimuta/protolint:latest
entrypoint: [""]
script:
- ls
- protolint lint .
only:
- main
- merge_requests The root cause is that GitLab CI requires images to have To show this workaround for GitLab CI, I'm going to add a note at https://github.com/yoheimuta/protolint#ci-integration. |
Great! Thank you for your time, I will test that! |
[memo] I found out that the image has
Consequently, I presume that running |
While running
yoheimuta/protolint:latest
in GitLab CI, I am seeing the following issue:I can't see why this is a issue, I switched to
docker.io/library/golang:alpine
and before script I installed it :This works fine. There might be an issue with how the image is built.
The text was updated successfully, but these errors were encountered: