Skip to content

fix(sqlite): remove depends_on db, for #46 - #57

Merged
stasadev merged 1 commit into
mainfrom
20260820_stasadev_sqlite
Aug 20, 2026
Merged

fix(sqlite): remove depends_on db, for #46#57
stasadev merged 1 commit into
mainfrom
20260820_stasadev_sqlite

Conversation

@stasadev

@stasadev stasadev commented Aug 20, 2026

Copy link
Copy Markdown
Member

The Issue

ddev config --omit-containers=db
ddev dotenv set .ddev/.env.adminer --adminer-default-driver=sqlite --adminer-default-db=/mnt/ddev_app/test.sqlite
ddev add-on get ddev/ddev-adminer
ddev restart
...
Failed to restart demo-project: service "adminer" depends on undefined service "db": invalid compose project

How This PR Solves The Issue

We don't need depends_on, I would like to add it conditionally, but Docker Compose doesn't support that.

Manual Testing Instructions

ddev config --omit-containers=db
ddev dotenv set .ddev/.env.adminer --adminer-default-driver=sqlite --adminer-default-db=/mnt/ddev_app/test.sqlite
ddev add-on get ddev/ddev-adminer --pr 57
ddev restart
ddev exec sqlite3 test.sqlite "CREATE TABLE items(id INTEGER PRIMARY KEY, name TEXT); INSERT INTO items(name) VALUES ('alpha'), ('beta');"
ddev adminer

Automated Testing Overview

Release/Deployment Notes

@stasadev
stasadev merged commit 36e2696 into main Aug 20, 2026
2 checks passed
@stasadev
stasadev deleted the 20260820_stasadev_sqlite branch August 20, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant