10000 Make distinct() work with arrays of dicts by niklasj · Pull Request #751 · mongomock/mongomock · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Make distinct() work with arrays of dicts #751

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 1 commit into from
Feb 2, 2022

Conversation

niklasj
Copy link
Contributor
@niklasj niklasj commented Jan 31, 2022

Fixes #750.

@pcorpet
Copy link
Member
pcorpet commented Feb 1, 2022

Very nice fix. Thanks.

@pcorpet
Copy link
Member
pcorpet commented Feb 1, 2022

I'm not ure why the CI didn't run. Can you try to rebase and push again please?

@codecov
Copy link
codecov bot commented Feb 2, 2022

Codecov Report

Merging #751 (548a806) into develop (84e0faf) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #751   +/-   ##
========================================
  Coverage    95.07%   95.07%           
========================================
  Files           19       19           
  Lines         3734     3734           
========================================
  Hits          3550     3550           
  Misses         184      184           
Impacted Files Coverage Δ
mongomock/collection.py 92.36% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84e0faf...548a806. Read the comment docs.

@niklasj
Copy link
Contributor Author
niklasj commented Feb 2, 2022

Sure thing!

@pcorpet
Copy link
Member
pcorpet commented Feb 2, 2022

Perfect, let's merge!

@pcorpet pcorpet merged commit b507ae6 into mongomock:develop Feb 2, 2022
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.

distinct fails for array field with dicts
2 participants
0