Skip to content

Conversation

@reslene
Copy link
Contributor

@reslene reslene commented Nov 24, 2025

No description provided.

@reslene reslene requested a review from a team as a code owner November 24, 2025 10:30
@coderabbitai
Copy link

coderabbitai bot commented Nov 24, 2025

Walkthrough

Updated Helm chart README badges: bumped chart versions for cloudprem, console-v3, and portal; bumped app versions for console-v3 and portal; root README summary table updated accordingly. (≤50 words)

Changes

Cohort / File(s) Summary
Root README summary
README.md
Updated Helm chart table entries: cloudprem chart 3.19.1 → 3.20.0; console-v3 chart 2.7.1 → 2.8.0 and AppVersion v1.16.0 → v1.17.0; portal chart 2.7.1 → 2.8.0 and AppVersion v1.16.0 → v1.17.0.
Chart README badges
charts/cloudprem/README.md, charts/console-v3/README.md, charts/portal/README.md
Bumped Version badges: cloudprem 3.19.1 → 3.20.0; console-v3 2.7.1 → 2.8.0; portal 2.7.1 → 2.8.0. Bumped AppVersion badges for console-v3 and portal: v1.16.0 → v1.17.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2–5 minutes

  • Simple, homogeneous documentation badge updates; verify version strings and no accidental other edits.

Possibly related PRs

Suggested reviewers

  • BrieucCaillot
  • Dav-14

Poem

🐰
I hopped through badges, tidy and spry,
Bumped versions skyward—three charts fly,
Cloudprem, Console, Portal in tune,
README refreshed beneath the moon,
A little patch, a happy sigh.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether it relates to the changeset. Add a description explaining the changes, even briefly mentioning the Helm chart version updates and their purpose.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main changes in the changeset: updating console-v3 and portal charts to version 1.17.0. However, it omits the cloudprem chart update (3.20.0), which is also part of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/platform_ui_v1.17.0

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 122bbe8 and dec0731.

⛔ Files ignored due to path filters (4)
  • charts/cloudprem/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • charts/cloudprem/Chart.yaml is excluded by !**/*.yaml
  • charts/console-v3/Chart.yaml is excluded by !**/*.yaml
  • charts/portal/Chart.yaml is excluded by !**/*.yaml
📒 Files selected for processing (4)
  • README.md (1 hunks)
  • charts/cloudprem/README.md (1 hunks)
  • charts/console-v3/README.md (1 hunks)
  • charts/portal/README.md (1 hunks)
✅ Files skipped from review due to trivial changes (1)
  • charts/cloudprem/README.md
🚧 Files skipped from review as they are similar to previous changes (2)
  • charts/portal/README.md
  • charts/console-v3/README.md
🧰 Additional context used
🪛 LanguageTool
README.md

[style] ~13-~13: Using many exclamation marks might seem excessive (in this case: 10 exclamation marks for a text that’s 2962 characters long)
Context: ...tions, users, roles, and permissions. | Artifact HUB | | Portal | 2.8.0 | v1.17.0 | Formance Portal | Artifact HUB | | Regions | 3.1.1 | latest | Formance Private Regions Helm Chart | Artifact HUB | | Stargate | 0.8.1 | latest | Formance EE Stargate gRPC Gateway | [![Artifact HUB](https://img.shields.io/en...

(EN_EXCESSIVE_EXCLAMATION)

🔇 Additional comments (1)
README.md (1)

8-8: ✓ Version updates confirmed and aligned across all files.

All versions in the root README.md match their corresponding chart README files:

  • Cloudprem (line 8): Chart 3.20.0 matches charts/cloudprem/README.md
  • Console-V3 (line 10): Chart 2.8.0 & App v1.17.0 match charts/console-v3/README.md
  • Portal (line 14): Chart 2.8.0 & App v1.17.0 match charts/portal/README.md

The changes are correct, consistent with the PR objectives, and the Cloudprem upgrade resolves the prior critical concern.


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.

Copy link

@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: 2

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 02f8aca and 122bbe8.

⛔ Files ignored due to path filters (4)
  • charts/cloudprem/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • charts/cloudprem/Chart.yaml is excluded by !**/*.yaml
  • charts/console-v3/Chart.yaml is excluded by !**/*.yaml
  • charts/portal/Chart.yaml is excluded by !**/*.yaml
📒 Files selected for processing (4)
  • README.md (1 hunks)
  • charts/cloudprem/README.md (1 hunks)
  • charts/console-v3/README.md (1 hunks)
  • charts/portal/README.md (1 hunks)
🧰 Additional context used
🪛 LanguageTool
README.md

[style] ~13-~13: Using many exclamation marks might seem excessive (in this case: 10 exclamation marks for a text that’s 2962 characters long)
Context: ...tions, users, roles, and permissions. | Artifact HUB | | Portal | 2.8.0 | v1.17.0 | Formance Portal | Artifact HUB | | Regions | 3.1.1 | latest | Formance Private Regions Helm Chart | Artifact HUB | | Stargate | 0.8.1 | latest | Formance EE Stargate gRPC Gateway | [![Artifact HUB](https://img.shields.io/en...

(EN_EXCESSIVE_EXCLAMATION)

🔇 Additional comments (4)
charts/console-v3/README.md (1)

3-3: Version badges updated correctly.

Chart version and app version bumps for console-v3 are appropriate (2.7.1 → 2.8.0 and v1.16.0 → v1.17.0). The changes align with the PR objective.

charts/portal/README.md (1)

3-3: Version badges updated correctly.

Chart version and app version bumps for portal are appropriate (2.7.1 → 2.8.0 and v1.16.0 → v1.17.0). The changes align with the PR objective.

README.md (2)

10-10: Console-V3 table entry updated correctly.

The console-v3 entry reflects the version bump (2.7.1 → 2.8.0) and app version update (v1.16.0 → v1.17.0), consistent with charts/console-v3/README.md.


14-14: Portal table entry updated correctly.

The portal entry reflects the version bump (2.7.1 → 2.8.0) and app version update (v1.16.0 → v1.17.0), consistent with charts/portal/README.md.

@reslene reslene force-pushed the feat/platform_ui_v1.17.0 branch from 122bbe8 to dec0731 Compare November 24, 2025 12:22
@reslene reslene enabled auto-merge November 24, 2025 14:33
@reslene reslene added this pull request to the merge queue Nov 24, 2025
Merged via the queue into main with commit 733625e Nov 24, 2025
6 checks passed
@reslene reslene deleted the feat/platform_ui_v1.17.0 branch November 24, 2025 14:33
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