Skip to content

Conversation

jpvanegasc
Copy link
Contributor

This PR includes docker configuration files for setting up the project locally via docker-compose. Instructions on how to do so have also been added to the README.

@stefmolin
Copy link
Owner

Thanks for sending this @jpvanegasc! I haven't used Docker much, so I will need some time to test this.

Do you happen to know if this will affect the GitHub Codespaces setup in any way?

@jpvanegasc
Copy link
Contributor Author

@stefmolin No worries, let me know if you have any questions or comments!

As for the GitHub Codespaces, it shouldn't. I did a short smoke test and it seems that it still works fine.

Copy link
Owner

@stefmolin stefmolin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jpvanegasc – this worked on my machine. I did notice a Docker warning, and I also think the Docker instructions could be reworked so it is easier to follow.

@jpvanegasc
Copy link
Contributor Author

@stefmolin Made the changes you suggested; let me know if you have any more comments!

@jpvanegasc jpvanegasc requested a review from stefmolin July 23, 2024 23:38
@jpvanegasc jpvanegasc requested a review from stefmolin July 27, 2024 20:32
Copy link
Owner

@stefmolin stefmolin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again for this, @jpvanegasc!

@stefmolin stefmolin merged commit 2f6909f into stefmolin:main Jul 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants