Skip to content

feat: Add local development support without Docker#21

Merged
acwilan merged 2 commits intomainfrom
feature/local-dev
Aug 1, 2025
Merged

feat: Add local development support without Docker#21
acwilan merged 2 commits intomainfrom
feature/local-dev

Conversation

@acwilan
Copy link
Contributor

@acwilan acwilan commented Aug 1, 2025

Changes

  • Added LOCAL_DEV.md with comprehensive instructions for running the app without Docker
  • Updated Makefile with local development commands (local-backend, local-frontend, local-migrate)
  • Modified README.md to reference the new local development documentation
  • Exposed MySQL port 3306 in docker-compose.yml to allow local connections
  • Added proxy configuration to frontend/vite.config.ts to support API requests in local dev mode

Why

These changes make it easier for developers to run the application directly on their local machine without requiring Docker, which can be beneficial for:

  • Faster development iterations
  • Developers with limited system resources
  • Environments where Docker setup is problematic
  • Debugging certain issues that are difficult to troubleshoot in containers

The Docker-based workflow remains fully supported and unchanged.

@acwilan acwilan self-assigned this Aug 1, 2025
Copy link
Contributor

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

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

Other comments (1)
  • docker-compose.yml (18-19) Exposing the database port 3306 to the host machine is useful for local development, but could present a security risk if used in production environments. Consider adding a comment in the docker-compose.yml file indicating this is for development purposes only, or using environment variables to conditionally expose the port based on the environment.

💡 To request another review, post a new comment with "/windsurf-review".

Co-authored-by: windsurf-bot[bot] <189301087+windsurf-bot[bot]@users.noreply.github.com>
@acwilan acwilan merged commit 6d43595 into main Aug 1, 2025
3 checks passed
@acwilan acwilan deleted the feature/local-dev branch August 1, 2025 16:26
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