Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
This PR replaces our default dependency resolver with coursier, upgrades some libraries for
consistency (like getting to circe 0.9.1 and geotrellis 2.0.0-RC2), explicitly adds some deps that weren't
picked up transitively (scalaz), uses some doobie combinators where we previously had strings, and adds an explicit
COURSIER_CACHE
environment variable to docker-compose. More on that last bit in notes.Checklist
Notes
The
COURSIER_CACHE
environment variable is useful for sandboxing -- it's now possible, if you want to try out changing some dependencies without having to worry about your cache state affecting results. You can specify a differentCOURSIER_CACHE
and know that the result of your dependencychanges has nothing to do with prior state. Neat! I used it as a way to test library upgrades to see how
much work it would be to get them to work, then changed the cache back and didn't have to
do anything to get to my prior working state. I threw this variable into
docker-compose.yml
because Ithink we're more likely to edit it and wanted it somewhere more obvious than
.env
, but I'm open tocriticism/ergonomics feedback on that front.
Testing Instructions
.ivy
cache (rm -rf app-backend/project/.ivy/cache/
)api/update
Closes #3757