Skip to content

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

Merged
merged 11 commits into from
Jan 24, 2025
Merged

Conversation

akhil-sumologic
Copy link
Collaborator

Purpose of this pull request

This pull request...

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/SUMO-251934

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Dec 2, 2024
Copy link
Collaborator

@jpipkin1 jpipkin1 left a 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.)

@jpipkin1
Copy link
Collaborator

jpipkin1 commented Dec 9, 2024

@akhil-sumologic, can we merge this PR now?

Copy link
Collaborator

@amee-sumo amee-sumo left a 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.

@amee-sumo
Copy link
Collaborator

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.)

I fixed the table.

@akhil-sumologic akhil-sumologic added the do-not-merge Pull requests that should not be merged label Dec 10, 2024
@akhil-sumologic
Copy link
Collaborator Author

@akhil-sumologic, can we merge this PR now?

This will be released in January as part of the AWSO 2.11.0 update.

@akhil-sumologic akhil-sumologic changed the title Updated doc for AWS tag filters Updated doc for AWS tag filters and AWSO apps Dec 10, 2024
Copy link
Collaborator

@amee-sumo amee-sumo left a comment

Choose a reason for hiding this comment

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

LGTM

@akhil-sumologic akhil-sumologic removed the do-not-merge Pull requests that should not be merged label Jan 24, 2025
@akhil-sumologic akhil-sumologic added this pull request to the merge queue Jan 24, 2025
Merged via the queue into main with commit 77ade0c Jan 24, 2025
6 checks passed
@akhil-sumologic akhil-sumologic deleted the sumo_251934 branch January 24, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Contributor approved, listed in .clabot file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants