Skip to content

Add ublue-os to the rebase command#845

Open
jumpyvi wants to merge 1 commit into
projectbluefin:mainfrom
jumpyvi:fix-rebase-command-drivers
Open

Add ublue-os to the rebase command#845
jumpyvi wants to merge 1 commit into
projectbluefin:mainfrom
jumpyvi:fix-rebase-command-drivers

Conversation

@jumpyvi
Copy link
Copy Markdown

@jumpyvi jumpyvi commented May 18, 2026

The rebase command in the drivers doc is broken

https://discord.com/channels/1345470678408626206/1345470679365189694/1506071443782369340

Summary by CodeRabbit

  • Bug Fixes
    • Fixed the "Rebase to this release" command to correctly reference the official image repository path.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 19dc3ff7-8949-4488-81f9-77f4182e619b

📥 Commits

Reviewing files that changed from the base of the PR and between 586d367 and 2cd8eab.

📒 Files selected for processing (1)
  • src/components/DriverVersionsCatalog.tsx

📝 Walkthrough

Walkthrough

The rebase command generation in DriverVersionsCatalog.tsx is updated to reference container images from the ublue-os namespace on the GitHub Container Registry. The bootc switch command now targets ghcr.io/ublue-os/<image-name>:<tag> instead of ghcr.io/<image-name>:<tag>.

Changes

Container Registry Path Update

Layer / File(s) Summary
Container image registry namespace update
src/components/DriverVersionsCatalog.tsx
The rebaseCommandForTag function is updated to generate bootc switch commands that reference container images with the ghcr.io/ublue-os/ namespace prefix.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A single path, refined with care,
ublue-os now lives right there—
Through ghcr's namespace it shall soar,
One tiny change, but solid core. 🐰

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding 'ublue-os' to the image reference path in the rebase command to fix the broken command documented in drivers.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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