CI: make sure we log in for deployments (#1541)
Over at https://github.com/sourcecred/sourcecred/pull/1514#discussion_r365443659 I suggested adding `deploy: false` to be less surprising with the tags we're pushing to DockerHub. As mentioned in #1512, we actually need `publish: true` to log in with DockerHub.
This commit is contained in:
parent
90b77a85dd
commit
c1f123f352
|
@ -124,7 +124,6 @@ workflows:
|
||||||
- test
|
- test
|
||||||
- test_full
|
- test_full
|
||||||
- test_full_10
|
- test_full_10
|
||||||
deploy: false
|
|
||||||
image: sourcecred/sourcecred
|
image: sourcecred/sourcecred
|
||||||
tag: latest
|
tag: latest
|
||||||
cache_from: node:12,sourcecred/sourcecred:dev
|
cache_from: node:12,sourcecred/sourcecred:dev
|
||||||
|
|
Loading…
Reference in New Issue