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

[AutoPR] monitor/resource-manager #3762

Merged
merged 4 commits into from
Oct 4, 2018

Conversation

AutorestCI
Copy link
Contributor

Created to accumulate context: monitor/resource-manager

…r Metric Alert API (#3714)

* Generated from 09c5415be341f2a6816dafb7a504818a3c536b7e

Multi Resource Alert changes

* Generated from a88d7f44fc48e9ea0c1203b6bf10bac1b65111e4

Merge remote-tracking branch 'upstream/master'
@ghost ghost self-requested a review October 4, 2018 20:00
@@ -2,7 +2,7 @@
"name": "azure-arm-monitor",
"author": "Microsoft Corporation",
"description": "MonitorManagementClient Library with typescript type definitions for node",
"version": "4.2.0",
"version": "4.3.0",
Copy link

Choose a reason for hiding this comment

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

This should actually be a major version change because MetricCriteria inherited a required property from MultiMetricCriteria. This will break any current MetricCriteria objects that currently exist.

@kpajdzik kpajdzik merged commit 3c60fc1 into master Oct 4, 2018
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.

3 participants