Skip to content

chore(plugin-fees): update appVersion and image tag to 3.0.5#917

Merged
guimoreirar merged 2 commits intomainfrom
hotfix/fees-v3.0.5
Dec 30, 2025
Merged

chore(plugin-fees): update appVersion and image tag to 3.0.5#917
guimoreirar merged 2 commits intomainfrom
hotfix/fees-v3.0.5

Conversation

@guimoreirar
Copy link
Copy Markdown
Member

@guimoreirar guimoreirar commented Dec 30, 2025

Midaz Pull Request Checklist

Pull Request Type

  • Midaz
  • Plugin Access Manager
  • Plugin CRM
  • Reporter
  • Plugin Fees
  • Plugin BR PIX Direct JD
  • Otel Collector
  • Pipeline
  • Documentation

Checklist

Please check each item after it's completed.

  • I have tested these changes locally.
  • I have updated the documentation accordingly.
  • I have added necessary comments to the code, especially in complex areas.
  • I have ensured that my changes adhere to the project's coding standards.
  • I have checked for any potential security issues.
  • I have ensured that all tests pass.
  • I have updated the version appropriately (if applicable).
  • I have confirmed this code is ready for review.

Additional Notes

Obs: Please, always remember to target your PR to develop branch instead of main.

Summary by CodeRabbit

  • Chores
    • Released plugin-fees service version 3.0.5 and updated deployment defaults to use the new release.
  • Documentation
    • Updated chart/readme defaults to reflect new service and frontend image tags (release-related version bumps).

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

@guimoreirar guimoreirar self-assigned this Dec 30, 2025
@guimoreirar guimoreirar requested a review from a team as a code owner December 30, 2025 18:49
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 30, 2025

Walkthrough

Bumped plugin-fees Helm chart metadata and default image tags: updated Chart.yaml.appVersion from 3.0.4 → 3.0.5, updated the container image tag in charts/plugin-fees/values.yaml to 3.0.5, and adjusted README default tags (fees.image.tag → 3.0.5; frontend.image.tag → 3.0.0). No changes to chart templates, control flow, logic, or exported/public APIs.

Sequence Diagram(s)

(omitted)

Possibly related PRs

Suggested labels

doc

Suggested reviewers

  • ferr3ira-gabriel
  • gauchito91

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: updating appVersion and image tag to 3.0.5 for the plugin-fees chart, which aligns with the file modifications.
Description check ✅ Passed The description follows the repository template with the correct plugin selected and appropriate checklist items marked, though several non-critical items remain unchecked.
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 ebff977 and 4b883a7.

📒 Files selected for processing (1)
  • charts/plugin-fees/README.md
🔇 Additional comments (2)
charts/plugin-fees/README.md (2)

81-81: Documentation updates are accurate and consistent.

The README parameter table updates correctly reflect the version bumps: fees.image.tag → 3.0.5 and frontend.image.tag → 3.0.0, which align with the commit message and PR title.

Also applies to: 117-117


1-143: Note: PR metadata and branch target need attention before merge.

A few observations outside the code changes themselves:

  1. Branch target: The PR description notes a request to target the develop branch instead of main. Ensure the correct target branch is set before merge.

  2. Checklist items: The PR body checklist has multiple unchecked items (documentation, coding standards, security review, tests, version updates). For a straightforward version bump hotfix, some of these may not apply, but please clarify which items are intentionally skipped vs. pending.

  3. Incomplete file visibility: This review includes only the README.md changes. The AI summary mentions updates to Chart.yaml (appVersion) and values.yaml (image tag). Please verify those files are also correctly updated and consistent with the README changes.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6d8d823 and ebff977.

📒 Files selected for processing (2)
  • charts/plugin-fees/Chart.yaml
  • charts/plugin-fees/values.yaml
🔇 Additional comments (1)
charts/plugin-fees/Chart.yaml (1)

19-19: Version consistency looks good. The appVersion (3.0.5) in Chart.yaml matches the image tag in values.yaml, ensuring the deployment uses the correct container image.

Comment thread charts/plugin-fees/values.yaml
@guimoreirar guimoreirar merged commit fc4cdda into main Dec 30, 2025
4 checks passed
@guimoreirar guimoreirar deleted the hotfix/fees-v3.0.5 branch December 30, 2025 19:08
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.

1 participant