Skip to content

Conversation

@nschonni
Copy link
Contributor

@nschonni nschonni commented Dec 2, 2018

There are currently 2 folders with the different casing in https://github.com/Azure/azure-rest-api-specs/tree/master/specification/applicationinsights/resource-manager
These are the only files that showed up as diffs imediately when I enabled git config core.ignorecase false in order to change casing on some other files.
The data-plane folder is also lowercase, but since it is all lowercase it doesn't cause the same issue.

@openapi-portal-comment
Copy link

If you're a MSFT employee, click this link
to view this PR's validation status on our new OpenAPI Hub spec management tool.

@AutorestCI
Copy link

AutorestCI commented Dec 2, 2018

Automation for azure-sdk-for-go

Nothing to generate for azure-sdk-for-go

@AutorestCI
Copy link

AutorestCI commented Dec 2, 2018

Automation for azure-sdk-for-js

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-js#576

@AutorestCI
Copy link

AutorestCI commented Dec 2, 2018

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI
Copy link

AutorestCI commented Dec 2, 2018

Automation for azure-sdk-for-python

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-python#2022

@AutorestCI
Copy link

AutorestCI commented Dec 2, 2018

Automation for azure-sdk-for-node

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-node#4188

@AutorestCI
Copy link

AutorestCI commented Dec 2, 2018

Automation for azure-sdk-for-java

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-java#2171

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@nschonni
Copy link
Contributor Author

nschonni commented Dec 2, 2018

Looks like the automation can't handle the case change, but I'm not sure if it would be an issue after it lands

@praries880 praries880 merged commit d02b8b6 into Azure:master Dec 5, 2018
@nschonni nschonni deleted the casing-Microsoft.Insights branch December 5, 2018 21:56
TalluriAnusha pushed a commit to AsrOneSdk/azure-rest-api-specs that referenced this pull request Feb 6, 2019
* chore: Rename microsoft.insights to Microsoft.Insights

* fix: Casing on applicationinsights/resource-manager YAML metadata
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