Skip to content

Conversation

@fengzhou-msft
Copy link
Member

Description

vcrpy is only needed for testing, it should not be a dependency for the azure-cli package.

Testing Guide

Tested the packages built with this PR's changes in build_test branch: https://dev.azure.com/azure-sdk/public/_build/results?buildId=573786&view=results.

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.

@fengzhou-msft fengzhou-msft added this to the S177 milestone Oct 14, 2020
@fengzhou-msft fengzhou-msft changed the title {Pakcaging} Remove vcrpy as a dependency {Packaging} Remove vcrpy as a dependency Oct 14, 2020
@yungezz
Copy link
Member

yungezz commented Oct 14, 2020

hi @arrownj

Copy link
Member

@yungezz yungezz left a comment

Choose a reason for hiding this comment

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

does need to remove from requirements.py3.windows?

@yonzhan yonzhan requested a review from jsntcy October 14, 2020 10:32
@fengzhou-msft
Copy link
Member Author

does need to remove from requirements.py3.windows?

It was not in requirements.py3.windows before.

@fengzhou-msft fengzhou-msft merged commit e6e6a6d into Azure:dev Oct 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants