-
Notifications
You must be signed in to change notification settings - Fork 266
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
Conversation
…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 $all and $elemMatch query operator tests * Accept a single Query operator in $elemMatch * Remove variadic argument for $elemMatch operator to prevent ambiquity
$eq $gt $gte $in $lt $lte $ne $nin
* PHPLIB-1335 Add tests on Element Query Operators * Make $type query operator variadic
* PHPLIB-1337 Add tests on Geospatial Query Operators * Fix nested $geometry
* 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
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.
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.
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.
Minor comment regarding fetch-depth
, but LGTM!
Fix PHPLIB-1506
replace
rule incomposer.json