Skip to content

[Network] Convert PublicIP Create to use SDK#2294

Merged
tjprescott merged 2 commits intoAzure:masterfrom
tjprescott:ConvertPublicIp
Mar 1, 2017
Merged

[Network] Convert PublicIP Create to use SDK#2294
tjprescott merged 2 commits intoAzure:masterfrom
tjprescott:ConvertPublicIp

Conversation

@tjprescott
Copy link
Copy Markdown
Member

@tjprescott tjprescott commented Feb 27, 2017

#Addresses #2198 for PublicIP.

Breaking Change: Previously this command returned the format:
{
"publicIp": { new-public-ip-object }
}
and now it simply returns:
{ new-public-ip-object }

The created object is now wrapped with "publicIp" to avoid a breaking change. It will be removed when the networking API next changes.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 27, 2017

Codecov Report

Merging #2294 into master will decrease coverage by -0.07%.
The diff coverage is 86.66%.

@@            Coverage Diff             @@
##           master    #2294      +/-   ##
==========================================
- Coverage   72.91%   72.84%   -0.07%     
==========================================
  Files         441      425      -16     
  Lines       19377    19294      -83     
  Branches     2750     2740      -10     
==========================================
- Hits        14129    14055      -74     
- Misses       4329     4330       +1     
+ Partials      919      909      -10
Impacted Files Coverage Δ
...ure/cli/command_modules/network/_client_factory.py 96.8% <ø> (-3.2%)
...etwork/azure/cli/command_modules/network/custom.py 59.29% <100%> (+0.26%)
...k/azure/cli/command_modules/network/_validators.py 63.37% <100%> (+0.04%)
...work/azure/cli/command_modules/network/commands.py 98.97% <100%> (ø)
...twork/azure/cli/command_modules/network/_format.py 37.93% <60%> (+2.08%)
...re-cli-sql/azure/cli/command_modules/sql/params.py 97.19% <0%> (-2.81%)
...li-core/azure/cli/core/commands/template_create.py 42.63% <0%> (-2.33%)
...ure-cli-sql/azure/cli/command_modules/sql/_util.py 89.77% <0%> (-1.7%)
...zure-cli-sql/azure/cli/command_modules/sql/help.py 100% <0%> (ø)
... and 2 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 8fb6b56...fc9967e. Read the comment docs.

@tjprescott tjprescott closed this Mar 1, 2017
@tjprescott tjprescott reopened this Mar 1, 2017
@tjprescott tjprescott merged commit 1ead810 into Azure:master Mar 1, 2017
@tjprescott tjprescott deleted the ConvertPublicIp branch March 1, 2017 18:18
00Kai0 pushed a commit to 00Kai0/azure-cli that referenced this pull request Apr 7, 2021
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