Skip to content

PHPLIB-1506 Import mongodb/builder #1381

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 72 commits into from
Sep 9, 2024
Merged

Conversation

GromNaN
Copy link
Member

@GromNaN GromNaN commented Sep 9, 2024

Fix PHPLIB-1506

alcaeus and others added 30 commits October 13, 2023 14:24
…ical field names (mongodb#9)

Numerical field names can't be used with named argument, because even if they comes from an array unpacking, the numerical keys are recalculated as a sequence: https://3v4l.org/Gj7jC

Everywhere a query is expected, an single array can be passed instead of the variadic arguments.

* Tuned the accepted types for field query
* Fixed CS (newline after if) in generated code
* Use BSON round-trip to normalize pipelines for comparison
* Use Canonical Extended JSON
* Fix $regex comparison
* Add first,firstN,last,lastN operators and tests

* Add tests on $first and $last
* Add $all and $elemMatch query operator tests

* Accept a single Query operator in $elemMatch

* Remove variadic argument for $elemMatch operator to prevent ambiquity
* PHPLIB-1335 Add tests on Element Query Operators

* Make $type query operator variadic
* PHPLIB-1337 Add tests on Geospatial Query Operators

* Fix nested $geometry
alcaeus and others added 13 commits February 2, 2024 13:28
* PHPLIB-1273: Improve $switch syntax

* Restore previously deleted encoding type in schema

* Introduce dedicated interface for branches in $switch

* Remove optional parameter in encodeAsObject

* Use case operator throughout tests
Bumps [ramsey/composer-install](https://github.com/ramsey/composer-install) from 2.2.0 to 3.0.0.
- [Release notes](https://github.com/ramsey/composer-install/releases)
- [Commits](ramsey/composer-install@2.2.0...3.0.0)

---
updated-dependencies:
- dependency-name: ramsey/composer-install
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Allow to merge an array of stages into a pipeline

* Improve types

* Add test to create pipeline from array
@GromNaN GromNaN marked this pull request as ready for review September 9, 2024 12:14
@GromNaN GromNaN requested a review from a team as a code owner September 9, 2024 12:14
@GromNaN GromNaN requested a review from alcaeus September 9, 2024 12:14
@GromNaN GromNaN marked this pull request as draft September 9, 2024 12:17
@GromNaN GromNaN marked this pull request as ready for review September 9, 2024 13:27
Copy link
Member

@alcaeus alcaeus left a comment

Choose a reason for hiding this comment

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

Reviewed the tooling changes, but did not review all the config and generator logic. I trust you copied that over from the builder project without changes. Just have a couple of minor comments, but in general this looks good to go.

Copy link
Member

@alcaeus alcaeus left a comment

Choose a reason for hiding this comment

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

Minor comment regarding fetch-depth, but LGTM!

@GromNaN GromNaN enabled auto-merge September 9, 2024 15:10
@GromNaN GromNaN merged commit e5e6b61 into mongodb:master Sep 9, 2024
30 checks passed
@GromNaN GromNaN deleted the PHPLIB-1506 branch September 9, 2024 16:20
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