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

[AutoPR] monitor/resource-manager #2541

Closed
wants to merge 3 commits into from

Conversation

AutorestCI
Copy link
Contributor

Created to accumulate context: monitor/resource-manager

…2018-03-01 (#2498)

* Generated from f55f6ab4bf342187e73b81b2ac967bfafc63cab2

Update with review feedback

* Generated from 56e36aba5586ca28ffa15bf1e4ced88a73fdba25

review feedback corrections, typos

* Generated from 39e64c50ed648db6330d2c91ccf30beda754b46e

fix examples

* Generated from c7c883711c9ea38d5802e14942e5763f8307b607

spelling error in comment
tenantId?: string;
readonly authorization?: SenderAuthorization;
readonly claims?: { [propertyName: string]: string };
readonly caller?: string;
Copy link

Choose a reason for hiding this comment

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

Marking properties as readonly is a breaking change. Requires major version increment.

@ghost
Copy link

ghost commented Mar 27, 2018

I regenerated this PR locally using the latest swagger and the latest generator. I'm going to close this PR without merging it.

@ghost ghost closed this Mar 27, 2018
@ghost ghost removed the in progress label Mar 27, 2018
@ghost ghost deleted the restapi_auto_monitor/resource-manager branch March 27, 2018 22:55
This pull request was closed.
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.

2 participants