Skip to content

Conversation

@ildyria
Copy link
Member

@ildyria ildyria commented Dec 31, 2025

Summary by CodeRabbit

  • Chores

    • Enabled multi-architecture Docker builds (linux/amd64, linux/arm64) and added CI steps for QEMU setup and registry login to support multi-arch publishing.
    • Enhanced image metadata handling and automated publishing flow for consistent tags and registry authentication.
  • Documentation

    • Added upgrade notice for v7.0, updated deployment image references and navigation links, and adjusted header/formatting for clarity.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 31, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Updates enable multi-architecture Docker builds and DockerHub publishing, add OCI metadata to images, and adjust README content; also remove public/Lychee-front from CI path-ignore rules.

Changes

Cohort / File(s) Summary
CI/CD Workflow
.github/workflows/CICD.yml
Removed public/Lychee-front from path-ignore lists; added QEMU setup step before Buildx; added DockerHub login step; expanded image metadata inputs to include both the repository and ghcr.io/${{ github.repository }}; added multi-platform targets linux/amd64 and linux/arm64; clarified latest tag handling in flavor config.
Dockerfile metadata
Dockerfile
Added multiple OCI image labels (title, description, authors, vendor, URL, documentation, licenses, base image name) to base and production stages; build/runtime steps unchanged.
Documentation
README.md
Removed Docker shield line; added Lychee 7.0 upgrade notice and link; replaced Docker image reference with Lychee container page URL; updated wording and symbols in several links and headings; adjusted branch/edge wording.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 I hopped through workflows late at night,
Puffed up QEMU and set platforms right.
Labels stacked tidy on each Docker layer,
README polished with gentle care.
DockerHub doors swing open — what a sight!

Pre-merge checks

✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d2a7af0 and 4a70c7b.

📒 Files selected for processing (2)
  • Dockerfile
  • README.md

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.

@ildyria ildyria changed the base branch from master to avoid-race-condition December 31, 2025 02:56
@ildyria ildyria marked this pull request as ready for review December 31, 2025 02:56
@ildyria ildyria requested a review from a team as a code owner December 31, 2025 02:56
@ildyria ildyria merged commit 19792d6 into avoid-race-condition Dec 31, 2025
4 of 5 checks passed
@ildyria ildyria deleted the docker-update branch December 31, 2025 03:08
@codecov
Copy link

codecov bot commented Dec 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.11%. Comparing base (bf38642) to head (4a70c7b).
⚠️ Report is 11 commits behind head on avoid-race-condition.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants