Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[dependencies] Upgrade pymongo>4 and motor>3. #386

Open
alexandervaneck opened this issue Apr 30, 2024 · 3 comments 路 May be fixed by #387
Open

[dependencies] Upgrade pymongo>4 and motor>3. #386

alexandervaneck opened this issue Apr 30, 2024 · 3 comments 路 May be fixed by #387

Comments

@alexandervaneck
Copy link

Hi @lafrech 馃憢

Thank you for maintaining umongo, a product we maintain depends on it! 馃檱

Lately it's come to our attention that we're not using the latest pymongo version and this is due to umongo[motor] being pinned to motor>2,<3.

Are you aware of a specific reason why motor is pinned <3?
It started supporting pymongo>4 after 3.0 unfortunately.

If it helps I can open a PR to investigate motor 3.x support in umongo 馃檶

@alexandervaneck
Copy link
Author

alexandervaneck commented Apr 30, 2024

I now see that umongo (optionally) depends on txmongo which also doesn't support pymongo>4. 馃

twisted/txmongo#278 (comment)

Looks like this was discussed before: #381

alexandervaneck pushed a commit to alexandervaneck/umongo that referenced this issue Apr 30, 2024
alexandervaneck pushed a commit to alexandervaneck/umongo that referenced this issue Apr 30, 2024
@alexandervaneck
Copy link
Author

Alright - I opened a PR to allow motor>3 and all tests pass. I did not check compatibility with txmongo since it could be seen as unrelated to motor support for pymongo.

I look forward to a review 馃檱 Let me know how best to merge this.

@lafrech
Copy link
Collaborator

lafrech commented May 1, 2024

Hi. Than you for your interest in umongo.

I think it is good practice to pin major versions so that everything does break when a new major dependency version is published. I totally agree with adding motor >=3,<4 support. Thanks for the PR.

Indeed this is independent from txmongo support (#381).

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 a pull request may close this issue.

2 participants