-
Notifications
You must be signed in to change notification settings - Fork 18
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
Wait for ephemeral container #61
Wait for ephemeral container #61
Conversation
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
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.
Recommend using Act to run linter locally. There are some additional linting failures to cause your changes to fail upon merge.
act --job run-golangci
We should probably update our Github Actions to run the run-golangci
job on creation of PRs rather than upon merge to main
.
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
483874c
to
5af50ad
Compare
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
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.
Verified changes will now pass linting. LGTM!
Add option for waiting for ephemeral container to be running
closes #60