Skip to content
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] Update docstrings/samples #30811

Merged
merged 2 commits into from
Jun 27, 2023

Conversation

pvaneck
Copy link
Member

@pvaneck pvaneck commented Jun 17, 2023

This cleans up the samples and adds a new sample for async batch queries. A new code snippets file was added to cleanly organize code snippets for class docstrings. The literalinclude indentation was also fixied for existing docstring snippets to fix a rander issue on the learn website.

  • Also public instances of docs.microsoft.com were updated to learn.microsoft.com.

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

This cleans up the samples and adds a new sample for async batch
queries. A new code snippets file was added to cleanly organize code
snippets for class docstrings.

Signed-off-by: Paul Van Eck <[email protected]>
@pvaneck pvaneck force-pushed the monitor-docstring-snippets branch from c284d7c to 55075b5 Compare June 26, 2023 21:34
@pvaneck pvaneck requested a review from kristapratico June 26, 2023 22:26
Signed-off-by: Paul Van Eck <[email protected]>
@pvaneck
Copy link
Member Author

pvaneck commented Jun 27, 2023

@kristapratico Thanks for the reviews :)

@pvaneck pvaneck merged commit 4bbd687 into Azure:main Jun 27, 2023
@pvaneck pvaneck deleted the monitor-docstring-snippets branch June 27, 2023 00:39
@scottaddie scottaddie added Monitor Monitor, Monitor Ingestion, Monitor Query and removed Monitor - Query labels Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Monitor Monitor, Monitor Ingestion, Monitor Query
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants