Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated Dockerfile and docker-compose.yml #126

Open
wants to merge 1 commit into
base: fourth-iteration
Choose a base branch
from

Conversation

nkarl
Copy link

@nkarl nkarl commented Nov 18, 2024

The container config is severely outdated. I have updated the following:

  1. pulled many dependencies out of Dockerfile and moved them to docker-compose as environment variables.
    • This helps with dependency management for future courses.
  2. updated as many dependencies as possible with respect to regression.
    • I didn't want to deviate too much from the 4th iteration. As I work through the Plutus Pioneer Program I am not yet at the level to know which pinned packages in cabal.project are safe to update.
  3. organized the Dockerfile to build more smoothly in VSCode.
    • for example, it makes little sense to run cabal commands in the middle of pulling the dependencies. These commands should be left to the users/students when they begin Week 02 of the course.

@nkarl nkarl changed the title Updated Dockerfile and docker-compose.yml to use more recent tools. Updated Dockerfile and docker-compose.yml Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant