Skip to content

Update docker compose and env example#13

Merged
warengonzaga merged 4 commits intomainfrom
chore/docker-compose
Oct 2, 2025
Merged

Update docker compose and env example#13
warengonzaga merged 4 commits intomainfrom
chore/docker-compose

Conversation

@warengonzaga
Copy link
Member

@warengonzaga warengonzaga commented Oct 2, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Documentation

    • Refreshed environment template: clearer sections (Application, Redis, Webhook, Optional Development), Docker-first Redis default, explicit supported platforms, added UNTHREAD_WEBHOOK_SECRET guidance, and clarified NODE_ENV notes.

  • Chores

    • Docker Compose updates: upgraded Redis with persistence enabled, enhanced health checks, standardized service naming and build metadata, removed redundant REDIS_URL, and switched to an explicit bridged network.

@warengonzaga warengonzaga self-assigned this Oct 2, 2025
Copilot AI review requested due to automatic review settings October 2, 2025 20:35
Copilot AI reviewed Oct 2, 2025
View reviewed changes
Copy link
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 Docker Compose and environment configuration files to improve documentation, service definitions, and deployment practices. The changes modernize the setup by using local builds instead of external images, updating Redis version, and providing clearer configuration guidance.

  • Modernizes Docker Compose configuration by removing version specification and improving service definitions
  • Updates Redis from version 7 to 8-alpine with enhanced persistence and healthcheck settings
  • Replaces external image dependency with local build configuration for the webhook server

Reviewed Changes

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

File Description
docker-compose.yml Updates service configurations, networking, and documentation with local build setup
.env.example Enhances documentation structure and provides clearer configuration guidance

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

@coderabbitai
Copy link

coderabbitai bot commented Oct 2, 2025 •
edited
Loading

Caution

Review failed

The pull request is closed.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

📝 Walkthrough

Walkthrough

The change reorganizes environment config and Docker orchestration: .env.example is restructured with Docker-centric defaults and a new webhook secret, while docker-compose.yml upgrades Redis to 8 with AOF persistence, renames/adjusts the server service, strengthens healthchecks, and moves networking to an explicit bridge network.

Changes

Cohort / File(s) Summary of Changes
Environment template adjustments
./.env.example		 Reorganized header and sections (Application, Redis, Webhook, Optional Development); Redis URL default changed to a Docker host redis-webhook; removed local-dev Redis note; clarified supported platforms list; added UNTHREAD_WEBHOOK_SECRET and guidance; moved commented NODE_ENV block to bottom and clarified note.
Docker orchestration updates
./docker-compose.yml		 Removed top-level version; Redis image upgraded redis:7-alpine → redis:8-alpine, added container_name, enabled AOF persistence via command: redis-server --appendonly yes, and extended healthcheck params; service webhook-server renamed to server, added build context and container_name, removed REDIS_URL env, adjusted healthcheck timings; networks updated to use driver: bridge with explicit name (external: true → bridge driver + name).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit's high-level summary is enabled.
Title Check ✅ Passed Certainly, sir. The title succinctly captures the dual focus on updating the Docker Compose configuration and the environment example file, aligning precisely with the primary modifications in the pull request.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 875d56c and 470e3c2.

📒 Files selected for processing (1)
  • docker-compose.yml (2 hunks)

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

warengonzaga and others added 2 commits October 3, 2025 04:48
@warengonzaga
☕ chore: update docker-compose.yml
ae17ccd 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@warengonzaga
☕ chore: update docker-compose.yml
470e3c2 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@warengonzaga warengonzaga mentioned this pull request Oct 2, 2025
Closed
@warengonzaga warengonzaga changed the title update docker compose and env example Update docker compose and env example Oct 2, 2025
@warengonzaga warengonzaga merged commit cacd014 into main Oct 2, 2025
2 checks passed
@warengonzaga warengonzaga deleted the chore/docker-compose branch October 2, 2025 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@warengonzaga warengonzaga

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@warengonzaga