Skip to content

chore: drop the unused docs image build - #53

Merged
scarmuega merged 1 commit into
mainfrom
chore/drop-docs-image
Oct 21, 2026
Merged

scarmuega merged 1 commit into
mainfrom
chore/drop-docs-image

Conversation

@scarmuega

@scarmuega scarmuega commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Removes docs/Dockerfile and its docs/.dockerignore and .github/workflows/docs.yml, which build and push ghcr.io/txpipe/boros-docs:<sha> on pushes to main.

Nothing deploys that image. The boros docs are served by the centralized docs.txpipe.io site (txpipe/docs), which pulls this repository as a submodule and builds the content itself; boros.txpipe.io redirects to https://docs.txpipe.io/boros. The docs sources under docs/ stay.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Removed the automated workflow for building and publishing the documentation container image.
    • Removed the Docker configuration used to build and run the documentation application.
    • Removed Docker ignore rules for documentation builds.

The docs are served by the centralized docs.txpipe.io site (txpipe/docs),
which pulls this repository as a submodule and builds the content itself.
Nothing deploys ghcr.io/txpipe/boros-docs, so the docs Dockerfile and the
workflow that publishes it on every push to main are dead weight.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: bd7a0b75-8f01-4fca-a2f7-41f0fa812f6c
📥 Commits

Reviewing files that changed from the base of the PR and between 64bd443 and 74363d2.

📒 Files selected for processing: 3
  • .github/workflows/docs.yml
  • docs/.dockerignore
  • docs/Dockerfile
💤 Files with no reviewable changes: 3
  • docs/Dockerfile
  • docs/.dockerignore
  • .github/workflows/docs.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The pull request removes the documentation Docker build configuration and its GitHub Actions workflow. The workflow had built and pushed a commit-tagged image to GHCR for linux/amd64.

Changes: Documentation container

Layer / File(s) Summary
Remove the documentation image pipeline
.github/workflows/docs.yml, docs/Dockerfile, docs/.dockerignore
The Docker build and runtime configuration, Docker ignore rules, and workflow for building and pushing the documentation image were removed.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: mduthey

Merge Risk: ⚪ Minimal · up to 74363

No repository-controlled documentation or deployment workflow appears to depend on the removed image. The change is mergeable after normal checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: removing the unused documentation image build.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

✨ Finishing Touches: 💡 1
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@scarmuega
scarmuega merged commit a6ba23e into main Oct 21, 2026
2 of 3 checks passed
@scarmuega
scarmuega deleted the chore/drop-docs-image branch October 21, 2026 19:10
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