Skip to content

Comments

[HDInsight] Fix syntax error in description of subDomainSuffix parameter#6287

Merged
sergey-shandar merged 1 commit intoAzure:masterfrom
aim-for-better:FixCreateApplicationNotSupportSubDomainSuffix
Jun 15, 2019
Merged

[HDInsight] Fix syntax error in description of subDomainSuffix parameter#6287
sergey-shandar merged 1 commit intoAzure:masterfrom
aim-for-better:FixCreateApplicationNotSupportSubDomainSuffix

Conversation

@aim-for-better
Copy link
Member

@aim-for-better aim-for-better commented Jun 11, 2019

This PR is related to previous PR [HDInsight] Fix specs not support subdomainsuffix parameter which has been merged.

In previous PR there is a syntax error in the description of subDomainSuffix parameter, this PR fixed it.

Latest improvements:

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Contribution checklist:

  • I have reviewed the documentation for the workflow.
  • Validation tools were run on swagger spec(s) and have all been fixed in this PR.
  • The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

  • Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
  • adding/removing APIs.
  • adding/removing properties.
  • adding/removing API-version.
  • adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

  • If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
    Please follow the link to find more details on API review process.

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Jun 11, 2019

SDK Automation [Logs] (Generated from cee83a7)

Succeeded Go: Azure/azure-sdk-for-go [Logs] [Diff]
Failed Java: Azure/azure-sdk-for-java [Logs] [Diff]
Succeeded Python: Azure/azure-sdk-for-python [Logs] [Diff]
Succeeded JavaScript: Azure/azure-sdk-for-js [Logs] [Diff]

@AutorestCI
Copy link

AutorestCI commented Jun 11, 2019

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI
Copy link

AutorestCI commented Jun 11, 2019

Automation for azure-sdk-for-python

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

@AutorestCI
Copy link

AutorestCI commented Jun 11, 2019

Automation for azure-sdk-for-java

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-java#3285

@AutorestCI
Copy link

AutorestCI commented Jun 11, 2019

Automation for azure-sdk-for-go

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

@AutorestCI
Copy link

AutorestCI commented Jun 11, 2019

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#3833

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@sergey-shandar
Copy link
Contributor

@pulkittomar @wawon-msft please review.

@idear1203
Copy link
Contributor

Hi @sergey-shandar , @wawon-msft is no longer working on this part. The change looks good to me. Could you please help merge the PR? Thanks in advance.

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.

6 participants