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

github-actions: remove python2 tests #3603

Merged
merged 2 commits into from
Mar 12, 2021

Conversation

alltilla
Copy link
Collaborator

As python2 became EOL and as every syslog-ng packager switched to python3 (except some old platforms in OBS) we no longer feel the need to support python2.

We do not plan to remove the python2 related code yet, because it would take a lot of effort, but we can easily remove the python2 related tests from the CI. This means with future changes we might break python2, and we will not see it, but we do not plan to support it anyways.

Signed-off-by: Attila Szakacs attila.szakacs@oneidentity.com

As python2 became EOL and as every syslog-ng packager switched to
python3 (except some old platforms in OBS) we no longer feel the need
to support python2.

We do not plan to remove the python2 related code yet, because it would
take a lot of effort, but we can easily remove the python2 related
tests from the CI. This means with future changes we might break
python2, and we will not see it, but we do not plan to support it
anyways.

Signed-off-by: Attila Szakacs <attila.szakacs@oneidentity.com>
@kira-syslogng
Copy link
Contributor

Build SUCCESS

Copy link
Collaborator

@Kokan Kokan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please write a news entry, as with this a direct python2 support also dropped. Even if python works for a while.

Signed-off-by: Attila Szakacs <attila.szakacs@oneidentity.com>
@kira-syslogng
Copy link
Contributor

Build SUCCESS

@MrAnno MrAnno merged commit 842c031 into syslog-ng:master Mar 12, 2021
yifan19 pushed a commit to yifan19/syslog-ng that referenced this pull request Apr 16, 2021
As stated by syslog-ng#3603
python2 is EOL.

Fix all shebangs calling python instead of python3
except the tests.

(correcting lib/merge-grammar.py)
Signed-off-by: Joe Slater <joe.slater@windriver.com>
(adding the rest)
Signed-off-by: Yi Fan Yu <yifan.yu@windriver.com>
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