Skip to content

Update PostgreSQL configuration and add Docker Compose setup#104

Merged
mahata merged 4 commits intomainfrom
feat/docker-compose
Mar 13, 2026
Merged

Update PostgreSQL configuration and add Docker Compose setup#104
mahata merged 4 commits intomainfrom
feat/docker-compose

Conversation

@mahata
Copy link
Copy Markdown
Owner

@mahata mahata commented Mar 13, 2026

Change PostgreSQL port configuration and introduce a Docker Compose setup for easier database management during development.

Copilot AI review requested due to automatic review settings March 13, 2026 07:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the local/dev and E2E database setup to use Docker Compose and a non-default Postgres host port, plus seeds an E2E “test” database on container init.

Changes:

  • Replace README Docker-run instructions with Docker Compose up/down workflow.
  • Introduce docker-compose.yml + init SQL to create a test database.
  • Shift configured Postgres port defaults/sample values from 5432 to 5437 across app, Drizzle, and Playwright E2E config.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
README.md Switches dev DB setup instructions to Docker Compose commands.
playwright.config.ts Updates E2E webServer DB env vars (notably POSTGRES_PORT).
hono/db/index.ts Changes default DB connection port fallback.
drizzle.config.ts Changes default Drizzle CLI connection port fallback.
docker-entrypoint-initdb.d/create-test-db.sql Creates a test database on first Postgres init.
docker-compose.yml Adds a Postgres service with port mapping, volumes, and init scripts.
.env.sample Updates sample DB port to match the new Compose host port.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread playwright.config.ts Outdated
Comment thread docker-compose.yml Outdated
Comment thread hono/db/index.ts
const pool = new Pool({
host: process.env.POSTGRES_HOST || "localhost",
port: Number(process.env.POSTGRES_PORT) || 5432,
port: Number(process.env.POSTGRES_PORT) || 5437,
Comment thread drizzle.config.ts
dbCredentials: {
host: process.env.POSTGRES_HOST || "localhost",
port: Number(process.env.POSTGRES_PORT) || 5432,
port: Number(process.env.POSTGRES_PORT) || 5437,
Comment thread README.md
mahata and others added 3 commits March 13, 2026 16:26
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@mahata mahata merged commit 1a30145 into main Mar 13, 2026
2 checks passed
@mahata mahata deleted the feat/docker-compose branch March 13, 2026 08:13
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.

2 participants