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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix deprecated import collections #253

Merged
merged 1 commit into from
Sep 29, 2021

Conversation

metal3d
Copy link
Contributor

@metal3d metal3d commented Mar 1, 2021

Replace import collections to import collection.abc for Python >= 3.3

See issue #252

Reason for the change
See #252 for collections deprecation

Description
collections is deprecated, we need to import collections.abc

Note that black made some code fixes (indentation, import order...)

Checklist

References
Anything else related to the change e.g. documentations, RFCs, etc.

@metal3d
Copy link
Contributor Author

metal3d commented Mar 5, 2021

Afraid that no one continues to support this project...

gabor-boros
gabor-boros previously approved these changes Mar 5, 2021
@metal3d
Copy link
Contributor Author

metal3d commented Mar 6, 2021

👍🏻 @gabor-boros :)

@tirkarthi
Copy link

This was resolved in 2 to 3 branch : #187

rethinkdb/ast.py Outdated Show resolved Hide resolved
Replace import collections to import collection.abc for Python >= 3.3

See issue rethinkdb#252
@gabor-boros gabor-boros merged commit 424cff7 into rethinkdb:master Sep 29, 2021
@gabor-boros
Copy link
Member

@metal3d Thank you for the contribution! Although a bit late, but approved and merged!

@metal3d
Copy link
Contributor Author

metal3d commented Oct 1, 2021

👍🏻 no problem

@Inveracity
Copy link
Contributor

@gabor-boros Now that 3.10 has been released will you cut a new release, please? It would be much appreciated, thank you <3

@gabor-boros
Copy link
Member

@Inveracity a massive refactor coming soon. I'll open the whip PR probably by Wednesday

@Inveracity
Copy link
Contributor

Heya @gabor-boros I'm curious if there has been any progress on the above?
Also is there an official channel for talking about joining the efforts?
I'd be very happy to help

@gabor-boros
Copy link
Member

@Inveracity Yes, we had some progress, though we are not ready yet with the changes. Feel free to join to Discord or slack and ping me.

@srh srh mentioned this pull request Oct 6, 2023
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.

None yet

4 participants