Skip to content

fix(docs): update wrong titles in doc and update to more recent doc URLs #622

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix(docs): update wrong titles in doc and update to more recent doc URLs #622

wants to merge 3 commits into from

Conversation

airadier
Copy link
Collaborator

@airadier airadier commented Mar 26, 2025 •
edited
Loading

  • Fix wrong doc title and example for IBM Function notification data source
  • Fix wrong title and update URL for Fargate workload agent
  • Fix document for secure_posture_policy data source

@airadier airadier changed the title fix(monitor-data-source): wrong doc title and example for IBM Function notification fix(docs): update wrong titles in doc and update to more recent doc URLs Mar 26, 2025
francesco-racciatti
francesco-racciatti approved these changes Mar 27, 2025
View reviewed changes
Copy link
Contributor

@francesco-racciatti francesco-racciatti left a comment

Choose a reason for hiding this comment

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

LGTM for the fargate part
Thanks for addressing this!

pgcrooks-sysdig
pgcrooks-sysdig approved these changes Mar 27, 2025
View reviewed changes
Copy link
Contributor

@pgcrooks-sysdig pgcrooks-sysdig left a comment

Choose a reason for hiding this comment

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

Fargate LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@francesco-racciatti francesco-racciatti francesco-racciatti approved these changes

@pgcrooks-sysdig pgcrooks-sysdig pgcrooks-sysdig approved these changes

@michelevergnano michelevergnano Awaiting requested review from michelevergnano michelevergnano is a code owner

@dkutsanov dkutsanov Awaiting requested review from dkutsanov dkutsanov is a code owner

@antonio-cuomo antonio-cuomo Awaiting requested review from antonio-cuomo antonio-cuomo is a code owner

@dbonf dbonf Awaiting requested review from dbonf dbonf is a code owner

@marojor marojor Awaiting requested review from marojor marojor is a code owner

@gnosek gnosek Awaiting requested review from gnosek gnosek is a code owner

@kmvachhani kmvachhani Awaiting requested review from kmvachhani kmvachhani is a code owner

@rosenbloomb-sysdig rosenbloomb-sysdig Awaiting requested review from rosenbloomb-sysdig rosenbloomb-sysdig is a code owner

@ombellare ombellare Awaiting requested review from ombellare ombellare is a code owner

@miguelgordo miguelgordo Awaiting requested review from miguelgordo miguelgordo is a code owner

@ivanlysiuk-sysdig ivanlysiuk-sysdig Awaiting requested review from ivanlysiuk-sysdig ivanlysiuk-sysdig is a code owner

@daniel-almeida daniel-almeida Awaiting requested review from daniel-almeida daniel-almeida is a code owner

@jbainbridgesysdig jbainbridgesysdig Awaiting requested review from jbainbridgesysdig jbainbridgesysdig is a code owner

@chen-shmilovich-sysdig chen-shmilovich-sysdig Awaiting requested review from chen-shmilovich-sysdig

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@airadier @francesco-racciatti @pgcrooks-sysdig