Skip to content

build(deps): update mongo-driver to v2#211

Merged
fredbi merged 2 commits intogo-openapi:masterfrom
GaoJianAllen:master
Mar 6, 2026
Merged

build(deps): update mongo-driver to v2#211
fredbi merged 2 commits intogo-openapi:masterfrom
GaoJianAllen:master

Conversation

@GaoJianAllen
Copy link

Change type

Please select: 🆕 New feature or enhancement|🔧 Bug fix'|📃 Documentation update

Short description

Fixes

Full description

Checklist

  • I have signed all my commits with my name and email (see DCO. This does not require a PGP-signed commit
  • I have rebased and squashed my work, so only one commit remains
  • I have added tests to cover my changes.
  • I have properly enriched go doc comments in code.
  • I have properly documented any breaking change.

Copy link
Member

@fredbi fredbi left a comment

Choose a reason for hiding this comment

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

Thanks. I agree that we must upgrade. Good move.

@fredbi
Copy link
Member

fredbi commented Mar 2, 2026

@GaoJianAllen thanks for this update.

Could you please fix the linting issues :

Error: mongo.go:13:2: ST1019: package "go.mongodb.org/mongo-driver/v2/bson" is being imported more than once (staticcheck)
"go.mongodb.org/mongo-driver/v2/bson"

And sign your commit (e.g. git commit -s , simple email will suffice) according to our DCO?

@GaoJianAllen GaoJianAllen force-pushed the master branch 2 times, most recently from 13121db to aaed5f3 Compare March 6, 2026 06:45
Signed-off-by: Gao, Jian (Allen) <[email protected]>
@GaoJianAllen GaoJianAllen marked this pull request as ready for review March 6, 2026 06:52
@GaoJianAllen
Copy link
Author

@fredbi Please review again.

Signed-off-by: Gao, Jian (Allen) <[email protected]>
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.01%. Comparing base (536ea3f) to head (a78efe6).
⚠️ Report is 2 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
mongo.go 45.45% 5 Missing and 1 partial ⚠️

❌ Your patch status has failed because the patch coverage (45.45%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #211   +/-   ##
=======================================
  Coverage   85.01%   85.01%           
=======================================
  Files          14       14           
  Lines        2015     2015           
=======================================
  Hits         1713     1713           
  Misses        218      218           
  Partials       84       84           

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

@fredbi fredbi merged commit 8a8c741 into go-openapi:master Mar 6, 2026
20 of 21 checks passed
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