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

Force Python 3 usage for MsgParser analyzer #604

Closed
wants to merge 4 commits into from

Conversation

iwitz
Copy link
Contributor

@iwitz iwitz commented Feb 5, 2020

MsgParser works with Python 3 and doesn't work with Python2 in our environment ; this PR forces the use of Python 3. Python 2 isn't maintained since January 1st.

@dadokkio
Copy link
Contributor

dadokkio commented Feb 5, 2020

#361

@nadouani
Copy link
Contributor

nadouani commented Feb 6, 2020

Hello @dadokkio, is this PR supposed to be included in 2.4.0 release? since it's related to #361 ?

I would say yes.

@jeromeleonard jeromeleonard changed the base branch from master to develop February 8, 2020 17:09
jeromeleonard added a commit that referenced this pull request Feb 8, 2020
@jeromeleonard jeromeleonard added this to the 2.4.0 milestone Feb 8, 2020
@jeromeleonard jeromeleonard added scope:analyzer Issue is analyzer related category:enhancement Issue is related to an existing feature to improve labels Feb 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:enhancement Issue is related to an existing feature to improve scope:analyzer Issue is analyzer related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants