Skip to content

chore: update payment version - #408

Merged
fguery merged 2 commits into
mainfrom
chore/update_payment_3.4.1
Aug 12, 2026
Merged

chore: update payment version#408
fguery merged 2 commits into
mainfrom
chore/update_payment_3.4.1

Conversation

@fguery

@fguery fguery commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@fguery
fguery requested a review from a team as a code owner August 12, 2026 09:46
@coderabbitai

coderabbitai Bot commented Aug 12, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

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: 844a2036-85fe-4f96-aad7-de33a1b07b00

📥 Commits

Reviewing files that changed from the base of the PR and between 154218b and 303e4de.

⛔ Files ignored due to path filters (8)
  • 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/formance/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • charts/formance/Chart.yaml is excluded by !**/*.yaml
  • charts/membership/Chart.yaml is excluded by !**/*.yaml
  • charts/portal/Chart.yaml is excluded by !**/*.yaml
  • charts/regions/Chart.yaml is excluded by !**/*.yaml
📒 Files selected for processing (7)
  • README.md
  • charts/cloudprem/README.md
  • charts/console-v3/README.md
  • charts/formance/README.md
  • charts/membership/README.md
  • charts/portal/README.md
  • charts/regions/README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • charts/formance/README.md

Walkthrough

The PR updates Helm chart versions in the root chart table and chart READMEs. It also updates the documented v3.2 Payments version from v3.4.0 to v3.4.1 in the Formance and Regions chart READMEs.

Changes

Chart version documentation

Layer / File(s) Summary
Update documented chart releases
README.md, charts/cloudprem/README.md, charts/console-v3/README.md, charts/formance/README.md, charts/membership/README.md, charts/portal/README.md, charts/regions/README.md		 The root chart table and chart READMEs now show the updated Helm chart versions. Unchanged application versions remain documented.
Synchronize documented Payments versions
charts/formance/README.md, charts/regions/README.md		 Both README entries now document Payments version v3.4.1 for v3.2.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • formancehq/helm#395: Updates the same chart README version references and root chart table.
  • formancehq/helm#401: Updates chart documentation and v3.2 component versions in the same README files.
  • formancehq/helm#407: Updates Helm chart documentation and version references, including Formance and Regions.

Suggested reviewers: sylr

Poem

A rabbit checks each chart release,
README versions now match in peace.
Payments hops from four-oh to one,
Chart updates across the table are done.
A tidy burrow, review complete!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so it does not communicate the chart version updates or related documentation changes. Add a brief description that summarizes the payments version update and the related chart version bumps.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: updating the payments version and related chart versions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/update_payment_3.4.1

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.

@NumaryBot

NumaryBot commented Aug 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

✅ Approve — automated review

The prior chart-version cascade issue is resolved at HEAD. All changed lockfiles have corresponding chart bumps, propagated through cloudprem to formance, with generated documentation kept consistent.

No findings.

NumaryBot
NumaryBot reviewed Aug 12, 2026
View reviewed changes

@NumaryBot NumaryBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NumaryBot posted 1 new inline finding.

Summary: #408 (comment)

Comment thread charts/regions/values.yaml
fabrice guery
chore: cascade chart version bumps for payments 3.4.1
303e4de 
Addresses review feedback on #408: lockfile changes were committed without
bumping the owning charts, so consumers would never receive them as releases.

Charts bumped:
- regions 3.15.0 -> 3.15.1 (payments v3.4.0 -> v3.4.1 in values.yaml)
- membership 3.6.2 -> 3.6.3 (postgresql 18.8.6 -> 18.8.8)
- portal 3.7.1 -> 3.7.2 (postgresql 18.8.6 -> 18.8.8)
- console-v3 3.7.1 -> 3.7.2 (postgresql 18.8.6 -> 18.8.8)
- cloudprem 4.10.2 -> 4.10.3 (parent of membership/portal/console-v3)
- formance 1.18.1 (umbrella: postgresql, regions, cloudprem)

Constraint: Chart.lock digest is what pins a release, so any lock change
requires bumping the owning chart and every parent up to formance
Rejected: Revert the incidental postgresql 18.8.6 -> 18.8.8 lock refresh to
keep the PR narrow | hand-editing Chart.lock is forbidden and just pre-commit
re-resolves it on the next run
Confidence: high
Scope-risk: narrow
Directive: All steps are patch-level upstream (payments and postgresql), so
every chart bump is a patch. No appVersion changed -- the leaf app versions
track their own services, not the postgresql sub-dependency.
NumaryBot
NumaryBot reviewed Aug 12, 2026
View reviewed changes

@NumaryBot NumaryBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NumaryBot review complete: no remaining inline findings.

Resolved 1 stale NumaryBot review thread (1 fixed, 0 outdated).

Summary: #408 (comment)

Dav-14
Dav-14 approved these changes Aug 12, 2026
View reviewed changes

@Dav-14 Dav-14 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@fguery
fguery enabled auto-merge (squash) August 12, 2026 13:19
@fguery
fguery merged commit 87c6fe3 into main Aug 12, 2026
6 checks passed
@fguery
fguery deleted the chore/update_payment_3.4.1 branch August 12, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NumaryBot NumaryBot NumaryBot left review comments
@flemzord flemzord flemzord approved these changes
@Dav-14 Dav-14 Dav-14 approved these changes

Assignees

No one assigned

Labels

docs release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fguery @NumaryBot @flemzord @Dav-14