Skip to content

fix: upgrade go-libs to v5.6.1#1424

Open
flemzord wants to merge 1 commit into
release/v2.4from
feat/go-libs-v5-6-1-release-v2.4
Open

fix: upgrade go-libs to v5.6.1#1424
flemzord wants to merge 1 commit into
release/v2.4from
feat/go-libs-v5-6-1-release-v2.4

Conversation

@flemzord

Copy link
Copy Markdown
Member

Summary

Validation

  • go test ./...
  • go build ./...
  • go test github.com/formancehq/go-libs/v5/pkg/storage/bun/connect -run 'TestReadWriteValidateConnectTimesOut|TestReadOnlyResetSessionTimesOut' -count=1 -v\n- (cd tools/generator && go test ./... && go build ./...)\n- (cd tools/provisioner && go test ./... && go build ./...)\n- (cd deployments/pulumi && go build ./...)\n\ndeployments/pulumi full tests require the pulumi CLI; this local environment does not have it on PATH.

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (8)
  • deployments/pulumi/go.mod is excluded by !**/*.mod
  • deployments/pulumi/go.sum is excluded by !**/*.sum, !**/*.sum
  • go.mod is excluded by !**/*.mod
  • go.sum is excluded by !**/*.sum, !**/*.sum
  • tools/generator/go.mod is excluded by !**/*.mod
  • tools/generator/go.sum is excluded by !**/*.sum, !**/*.sum
  • tools/provisioner/go.mod is excluded by !**/*.mod
  • tools/provisioner/go.sum is excluded by !**/*.sum, !**/*.sum

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1accd73d-ffc7-4832-a759-3fa6476fd932

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/go-libs-v5-6-1-release-v2.4

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.

@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.25%. Comparing base (ee1aa8c) to head (4c77b39).

Additional details and impacted files
@@               Coverage Diff                @@
##           release/v2.4    #1424      +/-   ##
================================================
+ Coverage         81.16%   81.25%   +0.08%     
================================================
  Files               206      206              
  Lines             11261    11259       -2     
================================================
+ Hits               9140     9148       +8     
- Misses             1559     1560       +1     
+ Partials            562      551      -11     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@flemzord flemzord marked this pull request as ready for review June 22, 2026 19:03
@flemzord flemzord requested a review from a team as a code owner June 22, 2026 19:03
@flemzord flemzord enabled auto-merge (squash) June 22, 2026 19:03

@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.

✅ Approve — automated review

The diff only updates go-libs pins and the corresponding module sums/indirect dependency versions. I did not identify any discrete breakage introduced by these dependency changes in the checked-in module files.

No findings.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants