Skip to content
This repository has been archived by the owner on May 5, 2023. It is now read-only.

[AutoPR network/resource-manager] Merge Network-2018-02-01 to Master Networking Swagger Changes for Feb 2018 #2533

Closed
wants to merge 2 commits into from

Conversation

AutorestCI
Copy link
Contributor

Fixed operationIds for non-CRUD operations in ExpressRouteCrossConnections (#2720)
Copy link

@ghost ghost left a 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;
Copy link

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 };
Copy link

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 };
Copy link

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant