-
Notifications
You must be signed in to change notification settings - Fork 1.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
[monitor-query] Regenerate client with stable swaggers and add in includeVisualization option. #16058
Conversation
…Analytics services. Each reference is by git commit ID this time to prevent any accidental drift. Also, added in the includeRender header (via the `includeVisualization` option). Fixes Azure#15771
@KarishmaGhiya - main owner @maorleger - backup owner
/azp run js - monitor-query - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
@KarishmaGhiya - I've updated the CODEOWNERS file now to reflect that you're the primary owner of this package (with @maorleger in there as your backup). |
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.
@richardpark-msft I think we'll need to run rushx format
@KarishmaGhiya - the entire empire toppled for lack of newlines. Formatting done. |
/azp run js - monitor-query - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run js - monitor-query - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
Big update to use the latest stable swagger files for all of the Log Analytics services. Each reference is by git commit ID this time to prevent any accidental drift.
Added in the includeRender header (via the
includeVisualization
option).Fixes #15771