Skip to content

Conversation

@clue
Copy link
Owner

@clue clue commented Sep 13, 2025

@clue clue requested a review from Copilot September 13, 2025 21:45
Copy link

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

This PR modernizes the development workflow by replacing custom shell scripts and Makefile targets with Docker Compose, adding watch mode for automatic rebuilds during development.

  • Introduces Docker Compose configuration with Apache and Tailwind services
  • Replaces manual Docker commands and shell scripts with compose-based workflow
  • Adds file watching capabilities for automatic rebuilds on code changes

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
compose.yaml New Docker Compose configuration with Apache and Tailwind services, including watch mode
Dockerfile Multi-stage build setup for composer dependencies, Tailwind CSS, and Apache server
README.md Updated documentation to use Docker Compose commands instead of Make targets
Makefile Simplified by removing custom serve targets and using Docker Compose for Tailwind
.github/workflows/ci.yml Updated CI to use Docker Compose instead of custom shell scripts
tests/await.bash Removed custom wait script, replaced by Docker Compose's built-in waiting

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@clue clue merged commit 454cc2a into clue:main Sep 13, 2025
2 checks passed
@clue clue deleted the compose branch September 13, 2025 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant