Skip to content

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

Open
flemzord wants to merge 1 commit into
mainfrom
feat/go-libs-v5-6-1-main
Open

fix: upgrade go-libs to v5.6.1#1425
flemzord wants to merge 1 commit into
mainfrom
feat/go-libs-v5-6-1-main

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: 220e4adc-dc55-49c4-963a-457626dcbd3f

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-main

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 80.63%. Comparing base (97b1d20) to head (5aace81).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1425      +/-   ##
==========================================
- Coverage   80.81%   80.63%   -0.18%     
==========================================
  Files         206      206              
  Lines       11268    11268              
==========================================
- Hits         9106     9086      -20     
- Misses       1611     1616       +5     
- Partials      551      566      +15     

☔ 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 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 changes are limited to updating go-libs module pins and corresponding go.sum/go.mod transitive metadata. I did not find a discrete regression introduced by this patch.

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