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
Light: Remove Python 2 compatibility #4174
Light: Remove Python 2 compatibility #4174
Conversation
No news file has been detected. Please write one, if applicable. |
020e6ee
to
87bbabd
Compare
- with dropping Python 2 support from Light pathlib2 pip package will not needed anymore - instead of pathlib2 Python 3 native pathlib will be used Signed-off-by: Andras Mitzki <mitzkia@gmail.com>
- after dropping Python 2 support Path module will be available via pathlib package Signed-off-by: Andras Mitzki <mitzkia@gmail.com>
Signed-off-by: Andras Mitzki <mitzkia@gmail.com>
- in Python3 we do not need to convert Path objects to string Signed-off-by: Andras Mitzki <mitzkia@gmail.com>
- in Python3 we can use time.monotonic() Signed-off-by: Andras Mitzki <mitzkia@gmail.com>
87bbabd
to
a224fba
Compare
Signed-off-by: Andras Mitzki <mitzkia@gmail.com>
- this is only a style-check fix Signed-off-by: Andras Mitzki <mitzkia@gmail.com>
Build FAILURE |
@kira-syslogng test this please; |
Maybe I am wrong, but as I can see new devshell image could built with the changes: If this is true than I would say:
So the changes would work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed the changes in detail, approved.
It seems that we only need Python2 for the kira image now.
The main goal of this PR:
Additional tasks with this PR: