Closed dorranh closed 3 years ago
Gas costs: No change. Entrypoint sizes: No change. Test coverage: No change.
FYI - For some reason the caching doesn't seem to be working on this as expected. Going to try a couple of different configurations before submitting for review.
Gas costs: No change. Entrypoint sizes: No change. Test coverage: No change.
Merging this PR following discussions with @gkaracha earlier today.
Fixes #287 by building
+dev-container
and+cli
targets in our main Github Actions workflow. As with other images produced in this workflow, the Docker images built by CI are pushed to our image registry on commit to master only. New images have two tags, one being the commit sha from which they were built and the other being alatest
/master
tag which will be updated each time a new commit is made on master.As a part of this PR I have made a couple of tweaks to our Earthfile and poetry packaging to address an issue with incorrect local caching of apt package lists and irreproducible poetry environments due to the lack of a lockfile in
./client
.