Skip to content

Fixed capitalization of CommerceAdmin.json#1617

Merged
olydis merged 1 commit intoAzure:currentfrom
deathly809:current
Aug 31, 2017
Merged

Fixed capitalization of CommerceAdmin.json#1617
olydis merged 1 commit intoAzure:currentfrom
deathly809:current

Conversation

@deathly809
Copy link
Copy Markdown
Member

There is an issue where the capitalization of of the commerceAdmin.json does not match the capitalization reference in the readme.md resulting in an error when using AutoRest.

I have fixed this capitalization error and tested it against my fork. Before I was testing it locally against the file. Because Github is case sensitive and Windows is not I did not see the error.

@msftclas
Copy link
Copy Markdown

@deathly809,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@azuresdkciprbot
Copy link
Copy Markdown

Hi There,

I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:

File: specification/azsadmin/resource-manager/commerce/readme.md
Before the PR: Warning(s): 0 Error(s): 0
After the PR: Warning(s): 0 Error(s): 1

AutoRest Linter Guidelines | AutoRest Linter Issues

Send feedback and make AutoRest Linter Azure Bot smarter day by day!

Thanks for your co-operation.

@olydis olydis merged commit 21a810b into Azure:current Aug 31, 2017
@AutorestCI
Copy link
Copy Markdown

No modification for AutorestCI/azure-sdk-for-python

@AutorestCI
Copy link
Copy Markdown

No modification for AutorestCI/azure-sdk-for-ruby

@AutorestCI
Copy link
Copy Markdown

No modification for AutorestCI/azure-sdk-for-node

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