Skip to content

typo: eventhub/resource-manager/Microsoft.EventHub/EventHub #4553

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
sarangan12 merged 1 commit into from
Nov 26, 2018
Merged

typo: eventhub/resource-manager/Microsoft.EventHub/EventHub #4553

sarangan12 merged 1 commit into from
Nov 26, 2018

Conversation

@nschonni
Copy link
Contributor

@nschonni nschonni commented Nov 24, 2018

  • successfuly -> successfully
  • vaule -> value
  • Autorization -> Authorization
  • Regenrate -> Regenerate
  • Messagning -> Messaging
  • envokes -> Invokes
  • neeeds -> needs
  • achive -> archive
  • anme -> name
  • Usermetadata -> User Metadata
  • reponse -> response
  • pairning -> pairing

Some of the example files had the typo in the names, if that is considered a breaking change I can remove those

@nschonni
typo: eventhub/resource-manager/Microsoft.EventHub/EventHub
6353a74 
- successfuly -> successfully
- vaule -> value
- Autorization -> Authorization
- Regenrate -> Regenerate
- Messagning -> Messaging
- envokes -> Invokes
- neeeds -> needs
- achive -> archive
- anme -> name
- Usermetadata -> User Metadata
- reponse -> response
- pairning -> pairing
@openapi-portal-comment
Copy link

openapi-portal-comment bot commented Nov 24, 2018

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 AutorestCI mentioned this pull request Nov 24, 2018
Merged
@AutorestCI
Copy link

AutorestCI commented Nov 24, 2018 •
edited
Loading

Automation for azure-sdk-for-python

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

@AutorestCI AutorestCI mentioned this pull request Nov 24, 2018
Merged
@AutorestCI
Copy link

AutorestCI commented Nov 24, 2018 •
edited
Loading

Automation for azure-sdk-for-js

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

@AutorestCI AutorestCI mentioned this pull request Nov 24, 2018
Merged
@AutorestCI
Copy link

AutorestCI commented Nov 24, 2018 •
edited
Loading

Automation for azure-sdk-for-node

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

@AutorestCI AutorestCI mentioned this pull request Nov 24, 2018
Merged
@AutorestCI
Copy link

AutorestCI commented Nov 24, 2018 •
edited
Loading

Automation for azure-sdk-for-ruby

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

@azuresdkci
Copy link
Contributor

azuresdkci commented Nov 24, 2018

Can one of the admins verify this patch?

@AutorestCI AutorestCI mentioned this pull request Nov 24, 2018
Merged
@AutorestCI
Copy link

AutorestCI commented Nov 24, 2018 •
edited
Loading

Automation for azure-sdk-for-go

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

@AutorestCI
Copy link

AutorestCI commented Nov 24, 2018 •
edited
Loading

Automation for azure-sdk-for-java

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

@AutorestCI AutorestCI mentioned this pull request Nov 24, 2018
Merged
sarangan12
sarangan12 approved these changes Nov 26, 2018
View reviewed changes
Copy link
Contributor

@sarangan12 sarangan12 left a comment

Choose a reason for hiding this comment

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

All description or example file name changes. Nothing impacts SDK.

@sarangan12 sarangan12 merged commit 273e8ad into Azure:master Nov 26, 2018
@nschonni nschonni deleted the typo-eventhub/resource-manager/Microsoft.EventHub/EventHub branch November 26, 2018 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@sarangan12 sarangan12 sarangan12 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@sarangan12 sarangan12

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nschonni @AutorestCI @azuresdkci @sarangan12