Closed jbencin closed 1 year ago
Merging #258 (06126f1) into master (a762f8f) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #258 +/- ##
=======================================
Coverage 93.47% 93.47%
=======================================
Files 6 6
Lines 337 337
=======================================
Hits 315 315
Misses 22 22
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Description
Push to DockerHub on branch commits, but only if we have credentials. This will make it easier to develop using
clarinet integrate
, which by default pulls Subnets from DockerHubAdditional info (benefits, drawbacks, caveats)
This PR reverts a change I made in #214, but only attempts the push to DockerHub if we have credentials, so we won't get CI failures when working in forked repos