Skip to content

Use docker compose to run database in development#81

Merged
Nudelsuppe42 merged 5 commits intoBuildTheEarth:api/v2from
SidneyNemzer:api-v2/db-dev
Mar 11, 2026
Merged

Use docker compose to run database in development#81
Nudelsuppe42 merged 5 commits intoBuildTheEarth:api/v2from
SidneyNemzer:api-v2/db-dev

Conversation

@SidneyNemzer
Copy link
Contributor

This change allows developers to run yarn dev and, assuming docker is installed, the database will start before running the api servers.

If you don't have docker, the dev scripts will fail. Let me know if you think docker is a reasonable dependency. I could make the new scripts exit early if docker is not detected.

This change allows developers to run `yarn dev` and, assuming docker is installed, the database will start before running the api servers
@Nudelsuppe42
Copy link
Contributor

Theoretically you dont need the individual scope#script scripts as turbo will automatically only run the script in projects where the script exists (eg. db:migrate only exists in @repo/db)

@SidneyNemzer
Copy link
Contributor Author

@Nudelsuppe42 Ah good point. Should I remove the scopes? I will say, it's nice that the scope makes it clear the script only runs in one package. But happy to remove them.

@Nudelsuppe42
Copy link
Contributor

What about putting frontend:dev with turbo run dev --filter=frontend... in the package.json? That way we only have the core scripts in turbo.

@SidneyNemzer
Copy link
Contributor Author

Added dev:frontend since we already have build:api and build:frontend. What do you mean by core scripts?

@Nudelsuppe42
Copy link
Contributor

I meant dev, build, etc.

@SidneyNemzer
Copy link
Contributor Author

Does this need any other changes or is it ready to merge you think?

Copy link
Contributor

@Nudelsuppe42 Nudelsuppe42 left a comment

Choose a reason for hiding this comment

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

Also, i'd remove the changes in turbo.json. The PG stuff looks good to me

@SidneyNemzer
Copy link
Contributor Author

SidneyNemzer commented Mar 10, 2026

Also, i'd remove the changes in turbo.json. The PG stuff looks good to me

You mean the scope prefixes? Just removed them. I think I can't totally remove the scripts from turbo.json because of the dependency order (up -> wait -> migrate -> dev)

@Nudelsuppe42 Nudelsuppe42 merged commit ee1c071 into BuildTheEarth:api/v2 Mar 11, 2026
1 check passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in @BuildTheEarth/web Tracker Mar 11, 2026
@Nudelsuppe42 Nudelsuppe42 added the docs Further information is requested label Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Further information is requested

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants