-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add a Development Container for a more seamless local experience #306
Add a Development Container for a more seamless local experience #306
Conversation
Hey ! Thank you for this! I didn't have my notifications turned on so I didn't see this PR. This all makes sense to me. I will look through this today |
No rush at all, thanks so much for taking a look! |
I think this approach is great. I didn't know about dev containers so this is a great new addition. Thank you! I couldn't quite get this one working (most likely my lack of understanding) but I assume with start up instructions that I will be able to 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works for me!
I'll update the README/documentation before merging. |
ab12f40
to
d587128
Compare
Hello!
While I was working on #305 and getting my local environment ready to go, I noticed that there were a lot of dependencies to get setup.
In order to make it easier to contribute, I thought it might be nice to have a way to run it all locally only relying on Docker and potentially VSCode if that is your editor of choice.
This Pull Request introduces a Dev Container for running the application without having to install any of the necessary dependencies on your machine. It uses the existing services defined in
docker-compose.yml
so that the services running are consistent.Here's what it looks like in VSCode.
Thanks to using Ubuntu Jammy as the base image, this can support ARM and Intel based machines.
If this is a path we'd like to move forward with, I'm happy to make changes to the README to show this as an option when getting started.
Thanks!