-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Fix docker_image ci job #4004
Fix docker_image ci job #4004
Conversation
909304d
to
dcbc009
Compare
Codecov Report
@@ Coverage Diff @@
## develop #4004 +/- ##
==========================================
+ Coverage 59.76% 59.8% +0.04%
==========================================
Files 211 199 -12
Lines 15055 14727 -328
==========================================
- Hits 8998 8808 -190
+ Misses 5435 5326 -109
+ Partials 622 593 -29 |
21fb42c
to
e34a766
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.
was just curl
missing?
docker login -u $DOCKER_USER -p $DOCKER_PASS | ||
docker push tendermint/gaia:$GAIAD_VERSION | ||
if [ -z "${GAIAD_VERSION}" ]; then | ||
docker build . |
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.
👍 seems healthy to fail always
We should add a pending log entry for a bugfix. |
- Always build regardless of the branch; push only from develop and master. - Add curl to packages dependencies in Dockerfile to allow golangci-lint to be downloaded. Closes: #3977
e34a766
to
887e98c
Compare
Done @alexanderbez |
push only from develop and master.
to allow golangci-lint to be downloaded.
Closes: #3977
docs/
)sdkch add [section] [stanza] [message]
Files changed
in the github PR explorerFor Admin Use: