search

hsanjuan / dccpi

A simple, easy to use, NMRA Digital Command Control (DCC) Python implementation for Raspberry Pi
GNU General Public License v3.0
54 stars 13 forks source link

updated dccpi setup to switch from python2.7 to python 3.8 #8

Closed samFabrice closed 3 years ago

samFabrice commented 3 years ago

I made an update of the version of python used by dccpi from python2.7 to python3.7 / python3.8

hsanjuan commented 3 years ago

These are just classifiers for pypi. Is there any code incompatibly that makes it not work with Python 3.8?

samFabrice commented 3 years ago

No. it is precisely because I tested it with python 3.7 and the dccpi API works that I thought it would be good to update its setup from python2.7 to python3.7.

In the API that I did and uses DCCPI, I use the pint library that exists only in Python3.7 what makes it clear from DCCPI I could not use mine

Le ven. 12 févr. 2021 à 18:21, Hector Sanjuan notifications@github.com a écrit :

These are just classifiers for pypi. Is there any code incompatibly that makes it not work with Python 3.8?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hsanjuan/dccpi/pull/8#issuecomment-778328160, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ625V6IDQSLUMICQXIUOTLS6VPTBANCNFSM4XRAT2GQ .

hsanjuan commented 3 years ago

setup.py says that it works with Python2.7 and Python 3, which should mean any Python3 version, including 3.7. But it also works with 2.7 still so no need to remove that.

samFabrice commented 3 years ago

Ok. I understood. thank you for your help

Le ven. 12 févr. 2021 à 20:10, Hector Sanjuan notifications@github.com a écrit :

setup.py says that it works with Python2.7 and Python 3, which should mean any Python3 version, including 3.7. But it also works with 2.7 still so no need to remove that.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hsanjuan/dccpi/pull/8#issuecomment-778392191, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ625V6BY66KMSAIEXQPKX3S6V4KTANCNFSM4XRAT2GQ .