Skip to content

Conversation

@AutorestCI
Copy link
Contributor

Generated from RestAPI PR: Azure/azure-rest-api-specs#1754

@AutorestCI AutorestCI requested a review from lmazuel as a code owner October 2, 2017 17:21
@lmazuel lmazuel changed the title Automatic PR from RestAPI-PR1754 [Resource/Policy] Add support for policy at ManagementGroup level Oct 2, 2017
@lmazuel lmazuel requested a review from vivsriaus October 2, 2017 17:23
@codecov-io
Copy link

codecov-io commented Oct 2, 2017

Codecov Report

Merging #1499 into master will decrease coverage by 0.04%.
The diff coverage is 5.4%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1499      +/-   ##
==========================================
- Coverage   55.07%   55.03%   -0.05%     
==========================================
  Files        3454     3454              
  Lines       84233    84307      +74     
==========================================
+ Hits        46395    46399       +4     
- Misses      37838    37908      +70
Impacted Files Coverage Δ
..._12_01/operations/policy_definitions_operations.py 54.08% <5.4%> (-42.39%) ⬇️

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 c9a8a87...868bdac. Read the comment docs.

@vivsriaus
Copy link

@lmazuel Can we please include changes from this spec PR as well?

@lmazuel
Copy link
Member

lmazuel commented Oct 4, 2017

@vivsriaus once the new spec PR will be merged, a new PR will be made automatically.

@vivsriaus
Copy link

@lmazuel Sorry I wasn't clear - I meant we should hold off on publishing a new package with the changes in this PR until the new spec PR is also merged.

return deserialized

def list_by_management_group(
self, management_group_id, filter=None, custom_headers=None, raw=False, **operation_config):

Choose a reason for hiding this comment

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

The filter param should go away with my changes in this PR.

@lmazuel
Copy link
Member

lmazuel commented Oct 5, 2017

Replaced by #1507

@lmazuel lmazuel closed this Oct 5, 2017
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