-
-
Notifications
You must be signed in to change notification settings - Fork 840
chore: add start-dev.sh for laziness #1628
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
Conversation
To me this would be useful. A few things I'm wondering:
|
mmm I haven't had to test anything that required it so i haven't thought about it lol.
If i need to dig deeper into the logs, I would run the separately, but usually I just want to start the dev environment to run a PR branch locally.
I was trying to get the volumes setup "nicely" and also getting the relative paths to the |
@xuatz Here's my understanding so far:
In summary, depending on what you're working on, you might need only one or the other, or both. If we had to choose only one, it would be difficult for me to pick which one would be more useful to have as a default. Otherwise, for consistency it seems appropriate to have both. |
@irobot |
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.
Awesome! Thank you!
This is pretty cool, thanks @xuatz! |
Summary
Details
Actually, i would prefer doing
docker compose up -d
, but it's not working perfectly at the moment. Even when it's perfect, there might be people who prefer not to use docker for dev, so maybe we can keep this even after that.But I'm not sure if we want this or not yet, but if this is welcomed, I'll update the docs as well to mention this.