Closed jbencin closed 1 year ago
Merging #262 (1729198) into master (3a0ef04) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #262 +/- ##
=======================================
Coverage 93.49% 93.49%
=======================================
Files 6 6
Lines 338 338
=======================================
Hits 316 316
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
Fixes the
docker push
step of CI by explicitly checking if a string is empty using!=
, rather than relying on an implicit castAdditional info (benefits, drawbacks, caveats)
This fixes an issue introduced by #258. If you check here, when I made these changes they worked fine. I'm not sure what changed since them to make it fail now.
Checklist
docs/rpc/openapi.yaml
andrpc-endpoints.md
for v2 endpoints,event-dispatcher.md
for new events)clarity-benchmarking
repobitcoin-tests.yml