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.

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

Fix Go SDK codegen for containerinstance #3735

Merged
merged 1 commit into from
Aug 27, 2018

Conversation

jhendrixMSFT
Copy link
Member

The package-2018-06 tag was incorrecly using the 2018-04 tag.
Moved Go SDK config section to its own config file.

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.

PR information

  • The title of the PR is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
  • Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
  • If applicable, the PR references the bug/issue that it fixes.
  • Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

The package-2018-06 tag was incorrecly using the 2018-04 tag.
Moved Go SDK config section to its own config file.
@AutorestCI
Copy link

AutorestCI commented Aug 27, 2018

Automation for azure-sdk-for-python

Nothing to generate for azure-sdk-for-python

These settings apply only when `--tag=package-2018-06 --go` is specified on the command line.
Please also specify `--go-sdk-folder=<path to the root directory of your azure-sdk-for-go clone>`.

``` yaml $(tag) == 'package-2018-04' && $(go)
Copy link
Member Author

Choose a reason for hiding this comment

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

Note that this uses the wrong tag, this is what was fixed in the new config file.

@AutorestCI
Copy link

AutorestCI commented Aug 27, 2018

Automation for azure-sdk-for-node

Nothing to generate for azure-sdk-for-node

@AutorestCI
Copy link

AutorestCI commented Aug 27, 2018

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI
Copy link

AutorestCI commented Aug 27, 2018

Automation for azure-sdk-for-java

Nothing to generate for azure-sdk-for-java

@AutorestCI
Copy link

AutorestCI commented Aug 27, 2018

Automation for azure-sdk-for-go

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

@jhendrixMSFT jhendrixMSFT merged commit 6cb8a5d into Azure:master Aug 27, 2018
@jhendrixMSFT jhendrixMSFT deleted the containerinstance branch August 27, 2018 23:18
ncalagar pushed a commit to ncalagar/azure-rest-api-specs that referenced this pull request Aug 28, 2018
* Remove TrackedResource from Swagger spec

"$ref": "../../common/v1/definitions.json#/definitions/TrackedResource"

* More updates on referring common types

More updates on referring common types

* Add UpdateParameters and Updateproperties

* Update example

* updated example

* update examples

Introducing new API for Azure Security Center Settings (Azure#3511)

* Add new API for Azure Security Center Settings

* Changed SettingsPUT API to receive parameter and not definition.

* - changed settings parameter name in example and in security parameter definition
- changes typo comment on settings get all example reference

* changes settings to setting in Settings API

* settings kind to setting kind in settings CSM API

Update batch resource-manager Node.js package version to 3.1.2 (Azure#3732)

IoTHub conf py (Azure#3733)

Reports API moved from stable to preview version with addition of documentation of a couple of new APIs (Azure#3640)

* Consumption specifications for cost insights and reporting operations

* Fixed $ref for ReportConfigDelete

* Silly me!! Fixed bad reference again

* Fixed schema validation issues

* Fixing validation errors around datetime and dynamic respones

* Fixed parameters in examples

* Fixed parameter name in example

* Removed x-ms-odata method for POST op

* Fixed ARM review comments

* Moved reports and usage analysis from microsoft.consumption to microsoft.costmanagament

* Merged with master again

* Updated reports api documentation

* Undo remove of API from stable version

providing description for new introduced property isRecurringCharge

fix example issues

Microsoft.ResourceGraph public preview spec (Azure#3716)

Add new microsoft.logic version (Azure#3683)

* Copy of the source version (2016-06-01) to seed the new version (2018-07-01-preview)

* Update Microsoft.Logic from 2016-06-01 to 2018-07-01-preview

* Remove extra prep files, update the readme

Fix Go SDK codegen for containerinstance (Azure#3735)

The package-2018-06 tag was incorrecly using the 2018-04 tag.
Moved Go SDK config section to its own config file.

Microsoft.NetApp first version (Azure#3639)

* Microsoft.NetApp first version

* Addressing pullrequest comments

added back billingperiod
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.

3 participants