Skip to content
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

[Hub Generated] Review request for Microsoft.Kusto to add version stable/2019-09-07 #7300

Merged
merged 2 commits into from
Sep 26, 2019
Merged

[Hub Generated] Review request for Microsoft.Kusto to add version stable/2019-09-07 #7300

merged 2 commits into from
Sep 26, 2019

Conversation

docohe
Copy link
Contributor

@docohe docohe commented Sep 24, 2019

If you are a MSFT employee you can view your work branch via this link.

Contribution checklist:

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Sep 24, 2019 •
edited
Loading

In Testing, Please Ignore

[Logs] (Generated from c1249a4, Iteration 2)

Warning .NET: test-repo-billy/azure-sdk-for-net [Logs] [Diff]
  • No packages generated.
Warning Python: test-repo-billy/azure-sdk-for-python [Logs] [Diff]
  • No packages generated.
Warning Go: test-repo-billy/azure-sdk-for-go [Logs] [Diff]
Failed Java: test-repo-billy/azure-sdk-for-java [Logs] [Diff]
  • Failed kusto/resource-manager/v2018_09_07_preview [Logs]
  • Failed kusto/resource-manager/v2019_01_21 [Logs]
  • Failed kusto/resource-manager/v2019_05_15 [Logs]
  • Failed kusto/resource-manager/v2019_09_07 [Logs]

@docohe docohe added the DoNotMerge <valid label in PR review process> use to hold merge after approval label Sep 24, 2019
@AutorestCI AutorestCI mentioned this pull request Sep 24, 2019
Merged
@AutorestCI
Copy link

AutorestCI commented Sep 24, 2019 •
edited
Loading

Automation for azure-sdk-for-python

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

@azuresdkci azuresdkci requested a review from yaohaizh September 24, 2019 12:47
@docohe
Copy link
Contributor Author

docohe commented Sep 24, 2019

We have presented this change a got ARM signed off here.

Please don't merge as long as the DoNotMerge tag exist.

@AutorestCI AutorestCI mentioned this pull request Sep 24, 2019
Merged
@AutorestCI
Copy link

AutorestCI commented Sep 24, 2019 •
edited
Loading

Automation for azure-sdk-for-go

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

@openapi-sdkautomation openapi-sdkautomation bot mentioned this pull request Sep 24, 2019
Closed
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@docohe docohe removed the DoNotMerge <valid label in PR review process> use to hold merge after approval label Sep 26, 2019
@docohe
Copy link
Contributor Author

docohe commented Sep 26, 2019

Please merge

@yaohaizh
Copy link
Contributor

yaohaizh commented Sep 26, 2019 •
edited
Loading

Originally approved PR from ARM: #5933

@yaohaizh yaohaizh merged commit bc3f39b into Azure:master Sep 26, 2019
@liatbezalel liatbezalel mentioned this pull request Oct 15, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yaohaizh yaohaizh yaohaizh approved these changes

Assignees

@yaohaizh yaohaizh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@docohe @AutorestCI @azuresdkci @yaohaizh