Skip to content

Use fakechris Docker namespace and publish multi-arch images#22

Merged
fakechris merged 1 commit intomainfrom
docs/dockerhub-fakechris-namespace
Apr 19, 2026
Merged

Use fakechris Docker namespace and publish multi-arch images#22
fakechris merged 1 commit intomainfrom
docs/dockerhub-fakechris-namespace

Conversation

@fakechris
Copy link
Copy Markdown
Owner

@fakechris fakechris commented Apr 19, 2026

Summary\n- Switch Docker image docs and compose defaults to fakechris namespace\n- Publish Docker images for linux/amd64 and linux/arm64 via QEMU/Buildx\n\n## Validation\n- ruby -e "require 'yaml'; YAML.load_file('.github/workflows/docker-publish.yml'); puts 'workflow yaml ok'"\n- docker compose -f docker-compose.images.yml config\n- docker compose --env-file .env.production.example -f docker-compose.prod.images.yml config\n\nNote: docs/handoff-droid.md is local untracked state and is not included.


Open in Devin Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 19, 2026

Warning

Rate limit exceeded

@fakechris has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 2 minutes and 0 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 2 minutes and 0 seconds.

⌛ 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.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b37b73ca-7687-40f4-8504-f773cf4b5cf2

📥 Commits

Reviewing files that changed from the base of the PR and between 2dec5a9 and 8fee905.

📒 Files selected for processing (4)
  • .github/workflows/docker-publish.yml
  • README.md
  • docker-compose.images.yml
  • docker-compose.prod.images.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/dockerhub-fakechris-namespace

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.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the default Docker image namespace from 'turnkeyai' to 'fakechris' across the README documentation and Docker Compose configuration files. The review feedback correctly identifies that several environment variable overrides in the README examples are now redundant because they match the new default values defined in the compose files, and suggests simplifying these commands for better readability.

Comment thread README.md
Comment on lines +145 to +146
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest pnpm compose:pull
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest pnpm compose:pull:up
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The environment variables INVOLUTE_IMAGE_NAMESPACE=fakechris and INVOLUTE_IMAGE_TAG=latest are now redundant in these examples because they match the default values defined in the updated docker-compose.images.yml and docker-compose.prod.images.yml files. Removing them simplifies the commands for users.

Suggested change
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest pnpm compose:pull
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest pnpm compose:pull:up
pnpm compose:pull
pnpm compose:pull:up

Comment thread README.md
Comment on lines +453 to 454
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest \
docker compose -f docker-compose.images.yml up -d db server web
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

These environment variables are redundant as they match the new defaults in the compose configuration. The command can be simplified by removing them.

Suggested change
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest \
docker compose -f docker-compose.images.yml up -d db server web
docker compose -f docker-compose.images.yml up -d db server web

Comment thread README.md
Comment on lines +460 to 462
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest \
docker compose --env-file .env.production \
-f docker-compose.prod.images.yml up -d
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

These environment variables are redundant as they match the new defaults in the compose configuration. Removing them makes the production example cleaner and easier to follow.

Suggested change
INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest \
docker compose --env-file .env.production \
-f docker-compose.prod.images.yml up -d
docker compose --env-file .env.production \
-f docker-compose.prod.images.yml up -d

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@fakechris fakechris merged commit 30942c5 into main Apr 19, 2026
3 checks passed
@fakechris fakechris deleted the docs/dockerhub-fakechris-namespace branch April 19, 2026 00:08
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