Skip to content

Conversation

@rajivnandivada
Copy link

No description provided.

@rajivnandivada rajivnandivada requested a review from lmazuel as a code owner May 10, 2019 21:32
@adxsdk6
Copy link

adxsdk6 commented May 10, 2019

Can one of the admins verify this patch?

1 similar comment
@adxsdk6
Copy link

adxsdk6 commented May 10, 2019

Can one of the admins verify this patch?

@codecov-io
Copy link

Codecov Report

Merging #5181 into master will increase coverage by 0.06%.
The diff coverage is 74.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5181      +/-   ##
==========================================
+ Coverage   48.69%   48.75%   +0.06%     
==========================================
  Files        6536     6560      +24     
  Lines      184777   185217     +440     
==========================================
+ Hits        89977    90305     +328     
- Misses      94800    94912     +112
Impacted Files Coverage Δ
...edservices/models/managed_services_client_enums.py 100% <100%> (ø)
...agedservices/models/registration_assignment_py3.py 100% <100%> (ø)
...agedservices/azure/mgmt/managedservices/version.py 100% <100%> (ø)
...e/mgmt/managedservices/models/authorization_py3.py 100% <100%> (ø)
...s/models/registration_definition_properties_py3.py 100% <100%> (ø)
...re/mgmt/managedservices/managed_services_client.py 100% <100%> (ø)
.../azure/mgmt/managedservices/operations/__init__.py 100% <100%> (ø)
...agedservices/models/registration_definition_py3.py 100% <100%> (ø)
...s/models/registration_assignment_properties_py3.py 100% <100%> (ø)
...gedservices/azure/mgmt/managedservices/__init__.py 100% <100%> (ø)
... and 38 more

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 6015ba4...0b39166. Read the comment docs.

@lmazuel lmazuel changed the base branch from master to restapi_auto_managedservices/resource-manager May 10, 2019 23:35
@lmazuel lmazuel changed the base branch from restapi_auto_managedservices/resource-manager to master May 10, 2019 23:37
@rajivnandivada
Copy link
Author

@lmazuel could you please review this? Hoping to get this merged soon..Thank you.

@lmazuel
Copy link
Member

lmazuel commented May 14, 2019

Hi @rajivnandivada , I cherry-picked your tests in #4676
Thanks!

@lmazuel lmazuel closed this May 14, 2019
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.

4 participants