-
Notifications
You must be signed in to change notification settings - Fork 5.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Hub Generated] Review request for Microsoft.Insights to add version stable/2018-03-01 #8412
[Hub Generated] Review request for Microsoft.Insights to add version stable/2018-03-01 #8412
Conversation
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-go - Release
|
azure-sdk-for-java - Release
|
azure-sdk-for-js - Release
|
azure-sdk-for-net - Release
|
azure-sdk-for-python - Release
|
@sivanguetta Please try to fix or suppress the model validation failure |
@NullMDR |
@akning-ms Please help to merge this PR as it's known issue in model validation. |
@akning-ms, can you please help to merge the PR? the merging is blocked due to a known issue in model validation. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Can one of the admins verify this patch? |
Hi @akning-ms and @raych1, |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
@sivanguetta , please fix this type of error |
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
Hi @raych1 |
@NullMDR , please help with prettiercheck failure. |
@sivanguetta Please rebase from latest master and try again |
…soft.Insights-2018-03-01-fix-documentation
Azure Pipelines successfully started running 1 pipeline(s). |
@sivanguetta Is this PR ready from your side? |
@NullMDR yes, the PR is ready from my side. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
azure-cli-extensions - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
@sivanguetta Azure pipeline running result will expire to save space. Anyway this PR LGTM. |
…stable/2018-03-01 (Azure#8412) * Update metric alert rules documentation * Fix READONLY_PROPERTY_NOT_ALLOWED_IN_REQUEST error * Fix Prettier issues * Fix Prettier issues
If you are a MSFT employee you can view your work branch via this link.
Contribution checklist: