-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: add goss testing for the docker image #4459
Conversation
4c07e71
to
8c13dd1
Compare
8c13dd1
to
172e34c
Compare
This PR is up to date and ready for review/merge :) |
Anything blocking this from going in? It would have detected a pretty significant regression in Atlantis, and a nice corner stone for further verification of the container images we build down the road :) |
time, is always time to review. |
Co-authored-by: Rui Chen <[email protected]>
Co-authored-by: Rui Chen <[email protected]>
@chenrui333 applied your suggested edits :) |
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
Co-authored-by: Rui Chen <[email protected]>
what
Adding automatic testing for the container image to avoid regressions like #4439 and #4458
why
Testing is good
tests
Running the tests in CI
references