-
-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use latest version of MongoDB API for v5.1+ compatibility #544
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just some comments about the index API usage
Updated with review, definitely better, thanks! |
7bbb97e
to
7074d1a
Compare
Added a few more places I originally missed, should be all good now. |
source/dubregistry/dbcontroller.d
Outdated
m_packages.ensureIndex([tuple("stats.score", 1)]); | ||
m_downloads.ensureIndex([tuple("package", 1), tuple("version", 1)]); | ||
IndexOptions opt; | ||
opt.unique = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
want to fix the mixed tabs and spaces here? otherwise LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes thx. Done.
Okay this needs a new version of Vibe.d to merge, will look into it today. |
you get my approval after dependency bump then |
@Geod24 got new version released yet? |
thanks for version bump @s-ludwig |
bc74711
to
770dead
Compare
Diff is pretty simple. Tested locally with MongoDB v6.0.6 on Ubuntu 22.04.
Needs a new release of Vibe.d to work.