Skip to content

[Packaging] drop support for python 3.5 - #12694

Merged
Feng Zhou (fengzhou-msft) merged 1 commit into
Azure:devfrom
fengzhou-msft:drop_python_3.5
Mar 25, 2020
Merged

[Packaging] drop support for python 3.5#12694
Feng Zhou (fengzhou-msft) merged 1 commit into
Azure:devfrom
fengzhou-msft:drop_python_3.5

Conversation

@fengzhou-msft

@fengzhou-msft Feng Zhou (fengzhou-msft) commented Mar 21, 2020

Copy link
Copy Markdown
Member

knack 0.7.0rc1 has dropped support for python 3.5.

Resolve #12525

History Notes:
(Fill in the following template if multiple notes are needed, otherwise PR title will be used for history note.)

[Component Name 1] (BREAKING CHANGE:) (az command:) make some customer-facing change.
[Component Name 2] (BREAKING CHANGE:) (az command:) make some customer-facing change.


This checklist is used to make sure that common guidelines for a pull request are followed.

@yonzhan

Copy link
Copy Markdown
Collaborator

drop python 3.5

@yungezz

Copy link
Copy Markdown
Member

hi Feng Zhou (@fengzhou-msft) why knack drop 3.5 support? what's impact to users if stopping support 3.5?

@fengzhou-msft

Copy link
Copy Markdown
Member Author

knack relies on mock 4.0 which dropped python 3.5. This impacts users using pip install with python 3.5. Telemetry shows the rate for python 3.5 in the past week is less than 0.5%.

Comment thread src/azure-cli/setup.py
with open('HISTORY.rst', 'r', encoding='utf-8') as f:
HISTORY = f.read()

if sys.version_info < (3, 4):

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why just remove this statement instead of changing it to if sys.version_info < (3, 6)?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same question.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This check was added in this PR. Adding this sets the hard limit for python versions at installation and block lower verisons. When we remove a version from the classifier, we're droppiong support for that version. It doesn't mean the code must not be compatible with that version. We can remove python 3.6 simply becaused it reaches end-of-life in the future, but our CLI code could still be totally compatible with python 3.6 at that time until at one point it's not. We will not fix it as we stated it's not supported.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Feng Zhou (@fengzhou-msft) I think I got your point, thanks for the clarification.

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.

az 2.2.0 does not appear to be compatible with Python 3.5

7 participants