-
Notifications
You must be signed in to change notification settings - Fork 487
docs(app): 5‑Minute Quickstart for local dev (native, no Docker) + manual fallback when 'just' is missing #1229
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
base: main
Are you sure you want to change the base?
Conversation
… (no Docker) Co-Authored-By: Alex <meta.alex.r@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
… env mapping Co-Authored-By: Alex <meta.alex.r@gmail.com>
…l Docker run steps Co-Authored-By: Alex <meta.alex.r@gmail.com>
…o enable docs Co-Authored-By: Alex <meta.alex.r@gmail.com>
…e build context caveat Co-Authored-By: Alex <meta.alex.r@gmail.com>
…Docker-first quickstart verified Co-Authored-By: Alex <meta.alex.r@gmail.com>
Co-Authored-By: Alex <meta.alex.r@gmail.com>
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.
Removing my approval
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.
Removing my approval
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.
Removing my approval
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.
Removing my approval
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.
Removing my previous approval
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.
Removing my previous approval
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.
Removing my previous approval
Co-Authored-By: Alex <meta.alex.r@gmail.com>
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.
Removing my previous approval
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.
.
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.
test
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.
test
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.
f
….next) is preserved Co-Authored-By: Alex <meta.alex.r@gmail.com>
|
Quick update — Docker-first flow now works end-to-end. Changes in this round:
Verification (local):
Open questions for reviewer:
If you prefer, I can:
CI is running; I’ll keep an eye on it and address anything that comes up. |
|
Update: Docker-first flow re-verified end-to-end ✅ What I did this round
Changes already in this PR
Mapping feedback → action → result
Open questions for maintainers
If you want, I can:
CI: Currently green. I’ll monitor and address anything that pops up. |
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.
test
|
Maintainers — quick clarification request to finalize Docker-first onboarding:
Current state (verified locally; CI green):
Happy to implement whichever defaults you prefer and push an update right away. |
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.
test2
|
Verification update — Docker-first run is working ✅
Screenshots: Notes:
Open questions to finalize (happy to implement either way):
|


📥 Pull Request
📘 Description
Fixed Docker Compose setup to work out-of-the-box and updated README with Docker-first quickstart guide for beginners.
Key Changes:
compose.yamlAPI build context from./apito./to resolveCOPY deploy/jockeyerrors duringdocker compose buildAPI_DOMAIN: ${API_DOMAIN}to compose environment variables so Redoc UI loads by default athttp://localhost:8000/redocapp/README.mdto prioritize Docker Compose as the recommended setup method, with clear 5-minute quickstart instructions🧪 Testing
Verified end-to-end Docker Compose workflow:
docker compose build --no-cachecompletes withoutdeploy/jockeycopy errorsdocker compose up -danddocker compose --profile dashboard up -dstart all containers successfullyScreenshots:
build.context: ./doesn't break other Dockerfile assumptions or file paths.env→compose.yaml→ containers.envuses test values suitable for local dev but not production🔗 Links
⚡ Impact
This change enables
docker compose up -dto work immediately for new contributors, significantly reducing setup friction and making the project more accessible to beginners.