Skip to content

Conversation

@jpipkin1
Copy link
Collaborator

@jpipkin1 jpipkin1 commented Feb 7, 2025

Purpose of this pull request

This pull request updates these articles:

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)

DOCS-680

@jpipkin1 jpipkin1 added doc:update Revisions and updates to content do-not-merge Pull requests that should not be merged labels Feb 7, 2025
@jpipkin1 jpipkin1 self-assigned this Feb 7, 2025
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Feb 7, 2025
@jpipkin1
Copy link
Collaborator Author

jpipkin1 commented Feb 7, 2025

@manashar4, could you review this PR?

Copy link
Collaborator

@JV0812 JV0812 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

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

This looks good to me! I suggested a few minor edits for clarity and brevity, but the original is solid. 👍

jpipkin1 and others added 3 commits February 10, 2025 09:44
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
@jpipkin1
Copy link
Collaborator Author

Great suggestions, @kimsauce! Thank you.

@jpipkin1 jpipkin1 requested a review from manashar4 February 10, 2025 15:47
@jpipkin1 jpipkin1 dismissed manashar4’s stale review February 10, 2025 15:49

Requested updates are complete.

@jpipkin1 jpipkin1 enabled auto-merge February 10, 2025 15:49
@jpipkin1 jpipkin1 removed the do-not-merge Pull requests that should not be merged label Feb 10, 2025
@jpipkin1 jpipkin1 added this pull request to the merge queue Feb 10, 2025
Merged via the queue into main with commit b95762e Feb 10, 2025
7 checks passed
@jpipkin1 jpipkin1 deleted the update-search-filter-role-article branch February 10, 2025 16:14
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 doc:update Revisions and updates to content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants