Skip to content

Conversation

gabor-boros
Copy link
Member

Reason for the change
#168

Description
Make the code Python 3.6+ compatible (and drop python2 support)

@gabor-boros gabor-boros added this to the Release 2.5.0 milestone Mar 18, 2020
@gabor-boros gabor-boros self-assigned this Mar 18, 2020
@gabor-boros
Copy link
Member Author

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 189
- Added 118
           

Coverage increased per file
===========================
+ rethinkdb/tornado_net/net_tornado.py  1
+ rethinkdb/errors.py  1
         

Coverage decreased per file
===========================
- rethinkdb/_dump.py  -1
- rethinkdb/net.py  -1
- rethinkdb/_export.py  -1
- rethinkdb/_index_rebuild.py  -1
         

Clones added
============
- rethinkdb/asyncio_net/net_asyncio.py  1
- rethinkdb/_dump.py  1
- rethinkdb/_export.py  1
- rethinkdb/trio_net/net_trio.py  1
- rethinkdb/ast.py  2
         

See the complete overview on Codacy

lsabi and others added 5 commits April 16, 2020 23:09
* Upgrade @asyncio.coroutine to async

* 🚧 2 tests do not pass yet

* 🚧 1 test error (nursery not found WTF) and 1 failing but don't know why..yet

* Remove commented code

* Use ImportError instead of Exception

Co-authored-by: lsabi <[email protected]>
Co-authored-by: Boros Gábor <[email protected]>
Import ABC from collections.abc for Python 3.
@ShadowJonathan
Copy link

wait, the package was not python3-ready?

why was that not excluded in the setup.py file with python_requires?

@ShadowJonathan
Copy link

@gabor-boros could you change python_requires=">=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*", in setup.py simply to python_requires=">=3.6"?

@gabor-boros
Copy link
Member Author

@ShadowJonathan The package is already python 3 compatible. This PR is about to make it Python 3 only. Whenever the PR is in that state, I'll make sure that setup.py has the right required python version.

@gabor-boros gabor-boros deleted the python-2-to-3 branch June 21, 2022 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants