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

chore: jsonfmt azurestack #5899

Merged
merged 1 commit into from
Jun 18, 2019
Merged

chore: jsonfmt azurestack #5899

merged 1 commit into from
Jun 18, 2019

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented May 9, 2019

Ran jsonfmt -w "specification/azurestack/**/*.json"

@nschonni
chore: jsonfmt azurestack
132764c 
Ran `jsonfmt -w "specification/azurestack/**/*.json"`
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@AutorestCI
Copy link

AutorestCI commented May 9, 2019 •
edited
Loading

Automation for azure-sdk-for-python

Nothing to generate for azure-sdk-for-python

@AutorestCI
Copy link

AutorestCI commented May 9, 2019 •
edited
Loading

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI AutorestCI mentioned this pull request May 9, 2019
Closed
@AutorestCI
Copy link

AutorestCI commented May 9, 2019 •
edited
Loading

Automation for azure-sdk-for-js

A PR has been created for you based on this PR content.

Once this PR will be merged, content will be added to your service PR:
Azure/azure-sdk-for-js#3918

@azuresdkci azuresdkci requested a review from shahabhijeet May 9, 2019 18:05
@AutorestCI
Copy link

AutorestCI commented May 9, 2019 •
edited
Loading

Automation for azure-sdk-for-go

Nothing to generate for azure-sdk-for-go

@AutorestCI AutorestCI mentioned this pull request May 9, 2019
Merged
@AutorestCI
Copy link

AutorestCI commented May 9, 2019 •
edited
Loading

Automation for azure-sdk-for-java

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

@shahabhijeet shahabhijeet requested a review from bganapa May 10, 2019 05:20
@shahabhijeet
Copy link
Member

@bganapa can you approve this PR

@shahabhijeet
Copy link
Member

Once CI passes, this PR is ready to be merged

@nschonni
Copy link
Contributor Author

Re-opening since CI looks stuck

@nschonni nschonni closed this May 15, 2019
@nschonni nschonni reopened this May 15, 2019
This was referenced May 15, 2019
Closed
Closed
@nschonni
Copy link
Contributor Author

@shahabhijeet looks like CI is happy again after re-opening

@nschonni
Copy link
Contributor Author

@shahabhijeet ping

@jhendrixMSFT jhendrixMSFT merged commit 4315e64 into Azure:master Jun 18, 2019
@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Jun 18, 2019 •
edited
Loading

SDK Automation [Logs] (Generated from 132764c)

Pending Python: Azure/azure-sdk-for-python
  • Package generation pending.
Pending Java: Azure/azure-sdk-for-java
  • Package generation pending.
Pending Go: Azure/azure-sdk-for-go
  • Package generation pending.
Pending JavaScript: Azure/azure-sdk-for-js
  • Package generation pending.

@AutorestCI AutorestCI mentioned this pull request Jun 18, 2019
Merged
@nschonni nschonni deleted the chore--jsonfmt-azurestack branch June 18, 2019 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bganapa bganapa bganapa approved these changes

@jhendrixMSFT jhendrixMSFT jhendrixMSFT approved these changes

@shahabhijeet shahabhijeet Awaiting requested review from shahabhijeet

Assignees

@shahabhijeet shahabhijeet

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@nschonni @azuresdkci @AutorestCI @shahabhijeet @jhendrixMSFT @bganapa