Skip to content

Adding swagger spec for MachineLearning/webservices RP#279

Closed
tudordid wants to merge 21 commits intoAzure:masterfrom
tudordid:master
Closed

Adding swagger spec for MachineLearning/webservices RP#279
tudordid wants to merge 21 commits intoAzure:masterfrom
tudordid:master

Conversation

@tudordid
Copy link
Copy Markdown
Contributor

No description provided.

tudordid and others added 21 commits March 23, 2016 16:04
Adapt swagger spec to latest RP wire contracts
- Add $skiptoken query parameter to list web services APIs
- removed the API definition for : /subscriptions/{subscriptionId}/providers/Microsoft/MachineLearning/checkNameAvailability as this is not an implemented API at this time
- made some corrections / additions to schema description elements
…arameters in the metod signature for generated code.
 - added title property to web service
 - changed API path for get keys API
 - changed StorageAccount details contract
 - renamed preview spec and updated to latest form in preparation to preview release
updated documentation on multiple items in the spec
made several items required as applicable
@azurecla
Copy link
Copy Markdown

Hi @tudordid, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (tudordid). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

"tags": [
"WebServices"
],
"operationId": "WebServices_CreateOrUpdateWebService",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We follow the Noun_Verb pattern for naming operationId.
This looks better WebServices_CreateOrUpdateWebService and will translate to client.WebServices.CreateOrUpdate()

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