Skip to content

Conversation

oseoin
Copy link
Contributor

@oseoin oseoin commented Mar 22, 2024

Proposed changes

Doc describing how to connect agent to security monitoring.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@github-actions github-actions bot added the documentation Pull requests/issues for documentation label Mar 22, 2024
@oseoin oseoin marked this pull request as ready for review March 22, 2024 13:47
@oseoin oseoin requested review from a team as code owners March 22, 2024 13:47
Copy link
Contributor

@ADubhlaoich ADubhlaoich left a comment

Choose a reason for hiding this comment

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

Generally LGTM, left some edit suggestions.

Co-authored-by: Alan Dooley <[email protected]>
Signed-off-by: oseoin <[email protected]>
@oseoin oseoin enabled auto-merge (squash) March 22, 2024 15:49
@oseoin oseoin disabled auto-merge March 22, 2024 16:28
@oseoin oseoin enabled auto-merge (squash) March 22, 2024 16:32
@oseoin oseoin merged commit b1ce3fa into main Mar 22, 2024
@oseoin oseoin deleted the security-monitoring-docs branch March 22, 2024 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests/issues for documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants