-
Notifications
You must be signed in to change notification settings - Fork 567
[AutoPR network/resource-manager] Merge Network-2018-02-01 to Master Networking Swagger Changes for Feb 2018 #2533
Conversation
Merge branch 'master' of https://github.com/Azure/azure-rest-api-specs into Network-2018-02-01
Fixed operationIds for non-CRUD operations in ExpressRouteCrossConnections (#2720)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requires major version increment.
* @member {string} [type] The resource type of the key vault. | ||
* @member {string} location The supported Azure location where the key vault | ||
* should be created. | ||
* @member {object} [tags] The tags that will be assigned to the key vault. | ||
*/ | ||
export interface Resource extends BaseResource { | ||
readonly id?: string; | ||
name: string; | ||
readonly name?: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marking an existing interface property as readonly is a breaking change. Requires major version increment.
@@ -2484,7 +2544,6 @@ export interface RouteFilterRule extends SubResource { | |||
name?: string; | |||
location?: string; | |||
readonly etag?: string; | |||
tags?: { [propertyName: string]: string }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing an interface property is a breaking change. Requires a major version increment.
*/ | ||
export interface PatchRouteFilterRule extends SubResource { | ||
access: string; | ||
communities: string[]; | ||
readonly provisioningState?: string; | ||
readonly name?: string; | ||
readonly etag?: string; | ||
tags?: { [propertyName: string]: string }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing an interface property is a breaking change. Requires major version increment.
Created to sync Azure/azure-rest-api-specs#2642