Skip to content

MongoDB: document changes in 2.0.0 #4618

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

Merged
merged 24 commits into from
Apr 17, 2025
Merged

Conversation

alcaeus
Copy link
Contributor

@alcaeus alcaeus commented Apr 15, 2025

This adds documentation changes for MongoDB 2.0. Documentation for methods and classes that were removed has been preserved - however I have removed deprecated constants and options for brevity.

Classes and methods that were removed in 2.0 will be removed from the docs at a future date.

@alcaeus alcaeus requested a review from GromNaN April 15, 2025 12:48
@alcaeus
Copy link
Contributor Author

alcaeus commented Apr 15, 2025

Note: I'm working on fixing the build error.

@alcaeus alcaeus force-pushed the mongodb-2.0.0-changes branch from 3da0758 to 37994a8 Compare April 15, 2025 17:57
Copy link
Member

@GromNaN GromNaN left a comment

Choose a reason for hiding this comment

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

LGTM.
I have some questions and remarks.

@@ -56,6 +56,7 @@
</row>
</thead>
<tbody>
&mongodb.changelog.tentative-return-types-enforced;
Copy link
Member

Choose a reason for hiding this comment

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

I think an example should be added to show that it is not necessary to add all return types with covariance. We can create a ticket to do this later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, we can consider doing this later. Let's see how many people struggle with the change.

Comment on lines +392 to +396
A negative value for the <literal>"limit"</literal> option no longer
implies &true; for the <literal>"singleBatch"</literal> option. To only
receive a single batch of results, combine a positive
<literal>"limit"</literal> value with the
<literal>"singleBatch"</literal> option.
Copy link
Member

Choose a reason for hiding this comment

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

I like the way you move the documentation of the removed option so people using a legacy version of the driver can still find their description even if the doc is not versioned.

@alcaeus alcaeus merged commit 4639881 into php:master Apr 17, 2025
2 checks passed
@alcaeus alcaeus deleted the mongodb-2.0.0-changes branch April 17, 2025 12:22
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.

3 participants