Skip to content

Conversation

@yamcodes
Copy link
Contributor

@yamcodes yamcodes commented Jan 17, 2026

Description

Remove devcontainers support


PR Checklist

  • Read the Developer's Guide in CONTRIBUTING.md
  • Use a concise title to represent the changes introduced in this PR
  • Provide a detailed description of the changes introduced in this PR, and, if necessary, some screenshots
  • Reference an issue or discussion where the feature or changes have been previously discussed
  • Add a failing test that passes with the changes introduced in this PR, or explain why it's not feasible
  • Add documentation for the feature or changes introduced in this PR to the docs; you can run them with bun docs

Summary by CodeRabbit

  • Chores
    • Removed development container environment configuration and setup specifications.
    • Removed automated database initialization and service readiness check scripts used in containerized development.
    • Removed documentation related to containerized development environment configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 17, 2026

📝 Walkthrough

Walkthrough

The PR removes the dev container infrastructure, including configuration file, documentation page, and database initialization scripts that were previously used for local development environment setup.

Changes

Cohort / File(s) Summary
Dev Container Configuration
\.devcontainer\.json
Deleted entire dev container configuration that defined Docker Compose setup with PostgreSQL, VS Code extensions, port forwarding, and lifecycle commands
Documentation
docs/dev-container\.md, docs/\.vitepress/config\.ts
Removed dev container documentation page and corresponding "Experimental" sidebar section reference
Database Scripts
scripts/create-db\.sh, scripts/wait-for-db\.sh
Deleted database initialization scripts that handled migration, seeding, and PostgreSQL readiness checks during dev container setup

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

  • #205 — Directly addresses removal of dev container feature and configuration infrastructure

Poem

🐰 The dev container hops away,
Scripts and configs cleared today,
Cleaner paths for those who stay,
Simpler setups on the way! 🌱

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Remove devcontainers' accurately and directly summarizes the main change: deletion of devcontainer configuration, documentation, and database setup scripts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@yamcodes yamcodes linked an issue Jan 17, 2026 that may be closed by this pull request
@yamcodes yamcodes merged commit 3e2ad52 into main Jan 17, 2026
8 checks passed
@yamcodes yamcodes deleted the remove-devcontainers branch January 17, 2026 13:28
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.

Dev containers - removed (context + related issues)

2 participants