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

{CI} Fix dependency version in setup.py, drop eoan package #16306

Merged
merged 2 commits into from
Dec 17, 2020

Conversation

fengzhou-msft
Copy link
Member

@fengzhou-msft fengzhou-msft commented Dec 17, 2020

Description

pip 20.3 has strict check for dependency versions. In CI the version is modified with a suffix of devN, this PR modifies the version requirements in setup.py to make it compatible in this case.

Ubuntu Eoan has reached end-of-life and is dropped in this PR.

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


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

@@ -53,7 +53,7 @@
'antlr4-python3-runtime~=4.7.2',
'azure-appconfiguration~=1.1.1',
'azure-batch~=10.0.0',
'azure-cli-core=={}.*'.format(VERSION),
'azure-cli-core=={}.*'.format(".".join(VERSION.split(".")[:3])),
Copy link
Member Author

Choose a reason for hiding this comment

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

VERSION can be in the format of 2.16.0 or 2.16.0.devN. Both cases will result in azure-cli-core==2.16.0.*.

Copy link
Member Author

Choose a reason for hiding this comment

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

It seems 2.16.0.* can match with both 2.16.0 and 2.16.0.devN, but 2.16.0.devN.* cannot match 2.16.0.devN in pip 20.3.

@fengzhou-msft fengzhou-msft changed the title {CI} Fix telemetry dependency version, drop eoan {CI} Fix dependency version in setup.py, drop eoan package Dec 17, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Dec 17, 2020

CI

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.

3 participants