Skip to content

feat: matrix ci - #2013

Merged
seefs001 merged 1 commit into
QuantumNous:alphafrom
seefs001:fix/ci
Oct 11, 2025
Merged

feat: matrix ci#2013
seefs001 merged 1 commit into
QuantumNous:alphafrom
seefs001:fix/ci

Conversation

@seefs001

@seefs001 seefs001 commented Oct 11, 2025

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Streamlined Docker image build and push to produce multi-arch (linux/amd64, linux/arm64) images in a single step.
    • Standardized image tagging and labels using metadata-driven tags.
    • Consolidated registry authentication into a unified login flow.
    • Upgraded Docker build action to the latest major version.
    • Removed per-architecture build matrix and separate manifest creation steps.
    • Simplified environment handling by eliminating legacy variables and cache configurations.

@seefs001
seefs001 merged commit aacdc39 into QuantumNous:alpha Oct 11, 2025
@coderabbitai

coderabbitai Bot commented Oct 11, 2025

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

Walkthrough

Workflow refactors Docker image build/push: switches from per-arch matrix and manifest creation to a single buildx build for linux/amd64 and linux/arm64 with metadata-derived tags/labels, full checkout, consolidated registry login, updated actions (build-push v5), and removal of legacy version/env handling and separate manifest jobs.

Changes

Cohort / File(s) Summary of Changes
CI: Docker image build/publish workflow
.github/workflows/docker-image-alpha.yml
Replaced per-arch matrix with single multi-platform build (amd64, arm64); uses docker/metadata-action for tags/labels; full checkout; earlier Buildx setup; unified registry login; switched to docker/build-push-action@v5; removed manifest-creation job and legacy env/version logic; static VERSION file usage for tagging context.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant GH as GitHub Event
  participant WF as Workflow Job
  participant R as Runner
  participant D as Docker Buildx
  participant Reg as Registries (Docker Hub/GHCR)

  GH->>WF: Trigger workflow
  WF->>R: actions/checkout (full)
  WF->>R: setup-buildx
  WF->>R: docker/metadata-action (derive tags/labels)
  WF->>Reg: docker/login-action (unified login)
  Note over WF,R: Platforms: linux/amd64, linux/arm64
  WF->>D: build-push-action@v5 build+push
  D->>Reg: Push image with metadata-derived tags
  Reg-->>WF: Push complete
  WF-->>GH: Job success
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

I hopped through YAML fields tonight,
Dropped extra matrices from my sight.
Two arches baked in one warm loaf,
Tags from whispers—metadata’s oath.
No manifest maze, just push and go—
Carrots cached? Not anymore—ho ho!
Ship it swift, ears in the flow.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 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 2842ae5 and 37cc5d2.

📒 Files selected for processing (1)
  • .github/workflows/docker-image-alpha.yml (1 hunks)

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.

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