Skip to content

chore: updated dynamic plugin version to 2.2.1#3947

Open
PreetiW wants to merge 1 commit into
redhat-developer:mainfrom
PreetiW:cost-plugin-version-2.2.1
Open

chore: updated dynamic plugin version to 2.2.1#3947
PreetiW wants to merge 1 commit into
redhat-developer:mainfrom
PreetiW:cost-plugin-version-2.2.1

Conversation

@PreetiW

@PreetiW PreetiW commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Hey, I just made a Pull Request!

chore: updated dynamic plugin version to 2.2.1

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-qodo-merge

Copy link
Copy Markdown

PR Summary by Qodo

Docs: bump cost-management dynamic plugin references to 2.2.1

📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Update cost-management dynamic plugin ConfigMap examples to reference image tag 2.2.1.
• Update backend plugin package example to 2.2.1 for consistent install instructions.
High-Level Assessment

The following are alternative approaches to this PR:

1. Use a version placeholder + single source of truth
  • ➕ Avoids repeated manual edits across docs when bumping versions
  • ➕ Reduces chance of frontend/backend example versions drifting
  • ➖ Requires docs tooling/conventions (templating, include files, or build-time substitution) that may not exist today
2. Document a floating tag strategy (e.g., 'latest' or 'stable')
  • ➕ Eliminates frequent doc bumps for patch releases
  • ➕ Simplifies install instructions for most users
  • ➖ Less reproducible installs; harder to debug user environments
  • ➖ Requires clear release/channel policy for the image registry

Recommendation: For now, this explicit patch-version bump is appropriate and keeps instructions reproducible. If these bumps are frequent across multiple docs, consider introducing a shared version placeholder/snippet to reduce maintenance and prevent drift.

Files changed (1) +2 / -2

Documentation (1) +2 / -2
dynamic-plugin.mdBump dynamic plugin package examples from 2.2.0 to 2.2.1 +2/-2

Bump dynamic plugin package examples from 2.2.0 to 2.2.1

• Updates the documented dynamic plugin 'package' image references for both the cost-management frontend and backend plugins to use the 2.2.1 tag. Keeps the installation examples aligned with the intended release.

workspaces/cost-management/docs/dynamic-plugin.md

@sonarqubecloud

Copy link
Copy Markdown

@rhdh-qodo-merge

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (1) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

Context used
✅ Compliance rules (platform): 27 rules
✅ Cross-repo context
  Not relevant to this PR: redhat-developer/rhdh
  Not relevant to this PR: redhat-developer/rhdh-chart
  Not relevant to this PR: redhat-developer/rhdh-operator
  Not relevant to this PR: redhat-developer/rhdh-local

Grey Divider


Informational

1. Missing changeset for doc update 📘 Rule violation § Compliance
Description
This PR modifies user-facing documentation (workspaces/cost-management/docs/dynamic-plugin.md) but
does not add a new .changeset/*.md entry for the affected workspace/packages. This can cause the
change to be omitted from release notes/versioning workflows.
Code

workspaces/cost-management/docs/dynamic-plugin.md[36]

+   - package: oci://quay.io/redhat-resource-optimization/dynamic-plugins:2.2.1!red-hat-developer-hub-plugin-cost-management
Relevance

⭐ Low

Precedent: changeset requests for Cost Management doc/README-only updates were rejected/ignored in
past PRs.

PR-#2679
PR-#2946

ⓘ Recommendations generated based on similar findings in past PRs

Evidence
PR Compliance ID 2217 requires a new .changeset/*.md entry when user-facing documentation is
modified. The diff updates the dynamic plugin documentation to reference version 2.2.1, but the PR
does not include any added changeset markdown file.

Rule 2217: User-facing changes must include a changeset markdown entry
workspaces/cost-management/docs/dynamic-plugin.md[36-36]
workspaces/cost-management/docs/dynamic-plugin.md[67-67]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
A user-facing documentation change was made, but no new Changeset markdown entry was added under `workspaces/cost-management/.changeset/`.

## Issue Context
Per repo policy, user-facing changes (including end-user documentation) must be accompanied by a `.changeset/*.md` file describing the change and affected packages.

## Fix Focus Areas
- workspaces/cost-management/docs/dynamic-plugin.md[33-74]
- workspaces/cost-management/.changeset/README.md[1-8]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@rhdh-qodo-merge rhdh-qodo-merge Bot added the documentation Improvements or additions to documentation label Jul 24, 2026
@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.34%. Comparing base (69b063b) to head (871dff9).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3947   +/-   ##
=======================================
  Coverage   57.34%   57.34%           
=======================================
  Files        2381     2381           
  Lines       95582    95582           
  Branches    26741    26748    +7     
=======================================
  Hits        54811    54811           
  Misses      39273    39273           
  Partials     1498     1498           
Flag Coverage Δ *Carryforward flag
adoption-insights 84.54% <ø> (ø) Carriedforward from 69b063b
ai-integrations 69.06% <ø> (ø) Carriedforward from 69b063b
app-defaults 69.79% <ø> (ø) Carriedforward from 69b063b
augment 46.67% <ø> (ø) Carriedforward from 69b063b
boost 75.83% <ø> (ø) Carriedforward from 69b063b
bulk-import 72.61% <ø> (ø) Carriedforward from 69b063b
cost-management 13.55% <ø> (ø)
dcm 60.72% <ø> (ø) Carriedforward from 69b063b
extensions 56.29% <ø> (ø) Carriedforward from 69b063b
global-floating-action-button 71.18% <ø> (ø) Carriedforward from 69b063b
global-header 62.17% <ø> (ø) Carriedforward from 69b063b
homepage 47.70% <ø> (ø) Carriedforward from 69b063b
install-dynamic-plugins 56.77% <ø> (ø) Carriedforward from 69b063b
intelligent-assistant 74.05% <ø> (ø) Carriedforward from 69b063b
konflux 91.98% <ø> (ø) Carriedforward from 69b063b
lightspeed 69.02% <ø> (ø) Carriedforward from 69b063b
mcp-integrations 83.40% <ø> (ø) Carriedforward from 69b063b
orchestrator 62.67% <ø> (ø) Carriedforward from 69b063b
quickstart 65.04% <ø> (ø) Carriedforward from 69b063b
sandbox 79.56% <ø> (ø) Carriedforward from 69b063b
scorecard 82.63% <ø> (ø) Carriedforward from 69b063b
theme 83.85% <ø> (ø) Carriedforward from 69b063b
translations 5.12% <ø> (ø) Carriedforward from 69b063b
x2a 79.31% <ø> (ø) Carriedforward from 69b063b

*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 69b063b...871dff9. 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation workspace/cost-management

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant