Make the code more modern

q1x / zabbix-gnomes

Various scripts to automate tasks in Zabbix

Other

179 stars 94 forks source link

Make the code more modern #7

Open hecko opened 6 years ago

hecko commented 6 years ago

migrate to python3
fix indentation
fix pyzabbix class init

arrfab commented 6 years ago

I'm not the author, but wondering why forcing a switch to python3 while quite a few distributions are still on python2. Using myself some of those scripts for the CentOS infra, and python 2.7 is still the default (and only one in base distro) for CentOs 7 (and let's not even mention python 2.6 on CentOS 6)

Of course that will be @q1x's decision, but that would mean that suddenly some people would have to revert those changes (into a different branch ?)

hecko commented 6 years ago

Well because Pyhton 2 might become depricated very quickly (https://pythonclock.org) and also because RHEL 8 will not suport Python 2 any more. Also these scripts are not primarily intended to be run regularly on the production infrastructure itself, rather run on dev systems where Im sure you can afford installing Python 3 from EPEL. Just my two cents.

geekpete commented 6 years ago

Can't it be made to run on both 2.7 and 3? https://docs.python.org/3/howto/pyporting.html

Many packages on Pypi work on both: https://python3wos.appspot.com/

ironbishop commented 4 years ago

is there anyone out there? python2 is deprecated by now