-
Notifications
You must be signed in to change notification settings - Fork 223
Updated doc for AWS tag filters and AWSO apps #4827
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
Conversation
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.
Approved, @akhil-sumologic. I made minor changes (see commit "Updates from review").
I can see why you added spaces to the table to make it display wider. But generally, we let the table width be set automatically. (See Tables in our style guide.)
@akhil-sumologic, can we merge this PR now? |
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.
LGTM. Just minor fixes.
I fixed the table. |
This will be released in January as part of the AWSO 2.11.0 update. |
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.
LGTM
Purpose of this pull request
This pull request...
Select the type of change
Ticket (if applicable)
https://sumologic.atlassian.net/browse/SUMO-251934