chore: Clean up Dockerfile and configuration

r-near commented 7 months ago

Renames compose.yaml to docker-compose.yml for better IDE support/naming convention
Simplify the Dockerfile. We're not taking advantage of dependency caching when doing prod deploys, so there's no point in adding all the extra complexity. Plus I think using a clean build for prod deploys is actually a good thing. Also, this builds fairly quickly (~1 minute) so I'm not too worried about it.

One thing to take into consideration is that if we're using Docker for local dev, this would slow things down.

anthony-near commented 7 months ago

I don't have too much Docker expertise, so I'm going to defer to @anshal-savla for this. I'm not using docker for local dev. Think it only makes sense to run docker locally to test before a release.

anshal-savla commented 7 months ago

LGTM @r-near why do you think using docker for local dev will slow you down?

r-near commented 7 months ago

LGTM @r-near why do you think using docker for local dev will slow you down?

Specifically adding a dependency will require rebuilding all dependencies from scratch. Personally I don't use Docker for local dev either.

near / pagoda-relayer-rs

chore: Clean up Dockerfile and configuration #80