Skip to content

Conversation

@ageorgou
Copy link
Contributor

Resolves #2644. This tightens the dependencies for three packages, so that they now require a version of azure-common that includes azure.profiles. More information in #2644.

Sorry if this is already being worked on!

@ageorgou ageorgou requested a review from lmazuel as a code owner May 30, 2018 10:34
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@msftclas
Copy link

msftclas commented May 30, 2018

CLA assistant check
All CLA requirements met.

@codecov-io
Copy link

codecov-io commented May 30, 2018

Codecov Report

Merging #2661 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2661   +/-   ##
=======================================
  Coverage   55.88%   55.88%           
=======================================
  Files        7161     7161           
  Lines      150710   150710           
=======================================
  Hits        84228    84228           
  Misses      66482    66482

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5f8a04...abca6ac. Read the comment docs.

Copy link
Member

Choose a reason for hiding this comment

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

Please remove this line that I already fixed on master and I will merge. Thanks!

The affected packages reference azure.profiles, which was added
to azure-common in version 1.1.9. This updates their requirements
accordingly.
@ageorgou ageorgou force-pushed the update-common-version-req branch from 2103959 to abca6ac Compare May 30, 2018 16:56
@ageorgou
Copy link
Contributor Author

Rebased after recent changes to master.

Copy link
Member

@lmazuel lmazuel left a comment

Choose a reason for hiding this comment

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

Awesome, thank you!

@lmazuel lmazuel merged commit b362f7f into Azure:master May 30, 2018
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.

5 participants