Skip to content

Add Owner to GraphRbac#1599

Closed
lmazuel wants to merge 3 commits intoAzure:masterfrom
lmazuel:graphrbac_owner
Closed

Add Owner to GraphRbac#1599
lmazuel wants to merge 3 commits intoAzure:masterfrom
lmazuel:graphrbac_owner

Conversation

@lmazuel
Copy link
Copy Markdown
Member

@lmazuel lmazuel commented Nov 2, 2017

Fix #1359

@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 2, 2017

Codecov Report

Merging #1599 into master will increase coverage by 0.03%.
The diff coverage is 96.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1599      +/-   ##
==========================================
+ Coverage   54.69%   54.73%   +0.03%     
==========================================
  Files        3680     3683       +3     
  Lines       90960    91049      +89     
==========================================
+ Hits        49750    49835      +85     
- Misses      41210    41214       +4
Impacted Files Coverage Δ
...re-graphrbac/azure/graphrbac/models/application.py 100% <100%> (ø) ⬆️
...c/azure/graphrbac/models/directory_object_paged.py 100% <100%> (ø)
azure-graphrbac/azure/graphrbac/models/ad_group.py 100% <100%> (ø) ⬆️
...aphrbac/models/application_add_owner_parameters.py 100% <100%> (ø)
azure-graphrbac/azure/graphrbac/models/user.py 100% <100%> (ø) ⬆️
...phrbac/azure/graphrbac/models/service_principal.py 100% <100%> (ø) ⬆️
...aphrbac/azure/graphrbac/models/directory_object.py 100% <100%> (ø)
azure-graphrbac/azure/graphrbac/models/__init__.py 100% <100%> (ø) ⬆️
...phrbac/operations/service_principals_operations.py 37.36% <90.47%> (+6.92%) ⬆️
...re/graphrbac/operations/applications_operations.py 39.9% <94.59%> (+11.49%) ⬆️
... and 3 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 bcc052b...024f714. Read the comment docs.

@lmazuel
Copy link
Copy Markdown
Member Author

lmazuel commented Jan 5, 2018

Replaced by #1724

@lmazuel lmazuel closed this Jan 5, 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.

2 participants