Skip to content

chore(app-defaults): add changeset for electron-to-chromium pin - #4205

Merged
nickboldt merged 1 commit into
mainfrom
chore/RHIDP-16097-app-defaults-changeset
Aug 7, 2026
Merged

nickboldt merged 1 commit into
mainfrom
chore/RHIDP-16097-app-defaults-changeset

Conversation

@nickboldt

Copy link
Copy Markdown
Member

Summary

  • Add patch changeset for the four app-defaults packages after #4203 (pin electron-to-chromium for hermetic Konflux).

Test plan

  • Changeset validation / CI green
  • Version Packages PR picks up the patch bumps

Generated-by: cursor

Ref: https://redhat.atlassian.net/browse/RHIDP-16097

Follow-up to #4203 so Version Packages can bump the app-defaults plugins.

https://redhat.atlassian.net/browse/RHIDP-16097

Generated-by: cursor
Co-authored-by: Cursor <cursoragent@cursor.com>

@rhdh-bot rhdh-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/lgtm
/approve

@rhdh-gh-app

rhdh-gh-app Bot commented Aug 7, 2026

Copy link
Copy Markdown

Unexpected Changesets

The following changeset(s) reference packages that have not been changed in this PR:

  • /home/runner/work/rhdh-plugins/rhdh-plugins/workspaces/app-defaults/.changeset/pin-electron-to-chromium.md: @red-hat-developer-hub/backstage-plugin-app-defaults, @red-hat-developer-hub/backstage-plugin-app-auth, @red-hat-developer-hub/backstage-plugin-app-integrations, @red-hat-developer-hub/backstage-plugin-app-react

Note that only changes that affect the published package require changesets, for example changes to tests and storybook stories do not require changesets.

@nickboldt

Copy link
Copy Markdown
Member Author

/override "check all required jobs"

@openshift-ci

openshift-ci Bot commented Aug 7, 2026

Copy link
Copy Markdown

@nickboldt: Overrode contexts on behalf of nickboldt: check all required jobs

Details

In response to this:

/override "check all required jobs"

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@rhdh-qodo-merge

Copy link
Copy Markdown

PR Summary by Qodo

Add changeset for app-defaults electron-to-chromium pin

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Add a patch changeset to bump four app-defaults packages after the electron-to-chromium pin.
• Ensure Version Packages automation picks up the intended patch releases.
• Document the rationale: avoid floating to newly published electron-to-chromium versions in
 hermetic Konflux builds.
Diagram

graph TD
  PR["PR change"] --> CS["changeset .md"] --> VP["Version Packages"] --> PUB["Published patch releases"]
  CS --> PKGS["4 app-defaults pkgs"]

  subgraph Legend
    direction LR
    _a["Automation"] ~~~ _f["File"] ~~~ _o["Outcome"]
  end
Loading
High-Level Assessment

Using a Changesets patch entry is the standard, lowest-risk way to ensure the Version Packages workflow publishes the required patch bumps for the app-defaults packages.

Files changed (1) +8 / -0

Other (1) +8 / -0
pin-electron-to-chromium.mdAdd patch changeset for four app-defaults packages +8/-0

Add patch changeset for four app-defaults packages

• Introduces a Changesets markdown file declaring patch bumps for four app-defaults-related packages. Includes context that the bumps are needed after pinning electron-to-chromium to keep Konflux/Hermeto builds hermetic and avoid proxy 404s from floating versions.

workspaces/app-defaults/.changeset/pin-electron-to-chromium.md

@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 7, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.33%. Comparing base (d89a157) to head (d97d268).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4205   +/-   ##
=======================================
  Coverage   58.33%   58.33%           
=======================================
  Files        2432     2432           
  Lines       96775    96775           
  Branches    26936    26936           
=======================================
  Hits        56456    56456           
  Misses      38860    38860           
  Partials     1459     1459           
Flag Coverage Δ *Carryforward flag
adoption-insights 84.55% <ø> (ø) Carriedforward from d89a157
ai-integrations 69.76% <ø> (ø) Carriedforward from d89a157
app-defaults 69.79% <ø> (ø)
augment 46.67% <ø> (ø) Carriedforward from d89a157
boost 76.77% <ø> (ø) Carriedforward from d89a157
bulk-import 72.79% <ø> (ø) Carriedforward from d89a157
cost-management 13.55% <ø> (ø) Carriedforward from d89a157
dcm 67.21% <ø> (ø) Carriedforward from d89a157
extensions 56.59% <ø> (ø) Carriedforward from d89a157
global-floating-action-button 71.18% <ø> (ø) Carriedforward from d89a157
global-header 66.50% <ø> (ø) Carriedforward from d89a157
homepage 47.59% <ø> (ø) Carriedforward from d89a157
install-dynamic-plugins 59.95% <ø> (ø) Carriedforward from d89a157
intelligent-assistant 75.24% <ø> (ø) Carriedforward from d89a157
konflux 91.98% <ø> (ø) Carriedforward from d89a157
lightspeed 69.02% <ø> (ø) Carriedforward from d89a157
mcp-integrations 83.40% <ø> (ø) Carriedforward from d89a157
orchestrator 66.91% <ø> (ø) Carriedforward from d89a157
quickstart 63.74% <ø> (ø) Carriedforward from d89a157
sandbox 79.56% <ø> (ø) Carriedforward from d89a157
scorecard 86.17% <ø> (ø) Carriedforward from d89a157
theme 88.77% <ø> (ø) Carriedforward from d89a157
translations 5.12% <ø> (ø) Carriedforward from d89a157
x2a 79.20% <ø> (ø) Carriedforward from d89a157

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d89a157...d97d268. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nickboldt
nickboldt merged commit 7f30033 into main Aug 7, 2026
34 checks passed
@rhdh-qodo-merge

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

To customize comments, go to the Qodo configuration screen, or learn more in the docs.

Qodo Logo

@rhdh-qodo-merge rhdh-qodo-merge Bot added the Other label Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants