Skip to content

Comments

rm docker-compose.yaml, docker/ (unused)#735

Open
mustafa0x wants to merge 3 commits intoquran:do-stagingfrom
mustafa0x:docker-cleanup
Open

rm docker-compose.yaml, docker/ (unused)#735
mustafa0x wants to merge 3 commits intoquran:do-stagingfrom
mustafa0x:docker-cleanup

Conversation

@mustafa0x
Copy link
Contributor

@mustafa0x mustafa0x commented Sep 29, 2025

Summary by CodeRabbit

  • Chores

    • Removed legacy container orchestration and build scripts.
    • Deleted obsolete environment and runtime configurations (infrastructure, logging, compression).
    • Cleaned up service-specific settings for backend, database, and search services.
    • No changes to application behavior or user-facing functionality.
  • Documentation

    • Removed outdated containerization and deployment documentation.

@coderabbitai
Copy link

coderabbitai bot commented Sep 29, 2025

Warning

Rate limit exceeded

@mustafa0x has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 12 minutes and 1 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between f0babbb and daecabb.

📒 Files selected for processing (4)
  • .byebug_history (0 hunks)
  • .stoplight (0 hunks)
  • .travis.yml (0 hunks)
  • README.md (1 hunks)

Walkthrough

This change removes local containerization and build tooling: the build script, all docker-compose files, multiple Docker-related configuration files (nginx, env, logrotate, gzip), and Docker-specific documentation. No replacements or new logic were added.

Changes

Cohort / File(s) Summary
Build script
build.sh
Deleted the build script that selected RAILS_ENV, built via docker-compose (dev/prod/test branches), managed a build-cache artifacts directory, and saved artifacts.
Compose orchestration
docker-compose.yml, docker-compose.production.yml, docker-compose.test.yml
Removed all docker-compose definitions for api, postgres, and elasticsearch across default, production, and test environments, including build args, images, env vars, volumes, and dependencies.
Docker runtime/config files
docker/backend.quran.com, docker/elasticsearch-env.conf, docker/gzip.conf, docker/nginx.logrotate.conf, docker/postgres-env.conf
Deleted nginx site config, Elasticsearch env passthroughs, gzip settings, nginx logrotate rules, and Postgres/secret env declarations.
Docker docs
docker/README.md
Removed Docker workflow documentation for development and production, including build steps, artifact caching, and run instructions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I tidied the burrow, swept out the tins,
Docker dreams folded with whiskery grins.
No more compose, no nginx logs—
Just lighter paws over fewer cogs.
Thump-thump! goes progress on cleaner ground,
Carrots deployed—no containers around. 🥕🐇

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 The title clearly indicates the removal of the Docker Compose file and the docker directory, which represents the bulk of the changeset’s deletions, and it succinctly conveys the primary action taken in the pull request.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

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.

@mustafa0x mustafa0x closed this Sep 29, 2025
@mustafa0x mustafa0x reopened this Oct 26, 2025
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