Skip to content
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

docs: fix issues in jq filter examples and broken links #2567

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

cmppoon
Copy link
Contributor

@cmppoon cmppoon commented Jun 17, 2024

I made some improvements in the docs. Please let me know if any changes are required. I hope it helps, thanks!

  1. Added missing closing quotes in some jq filter examples to correct syntax highlighting issues.

  2. Fixed markdown syntax to eliminate warnings.

  3. Fixed broken links for TrackSock and UntrackSock in hooks.md.

Thanks for contributing! Please ensure your pull request adheres to the following guidelines:

  • All commits contain a well written commit message and are signed-off (see Submitting a pull request).
  • All code is covered by unit and/or end-to-end tests where feasible.
  • All generated files are updated if needed (see Making changes).
  • Provide a title or release-note blurb suitable for the release notes (see guidelines).
  • Update documentation and write an upgrade note if needed (see guidelines).
  • Are you a user of Tetragon? Please add yourself to the Users doc in the Cilium repository.

Fixes: #issue-number

1. Added missing closing quotes in some jq filter examples to correct syntax highlighting issues.

2. Fixed markdown syntax to eliminate warnings.

3. Fixed broken links for TrackSock and UntrackSock in hooks.md.

Signed-off-by: Chaiyapruek Muangsiri <cmp.poon@gmail.com>
@cmppoon cmppoon requested review from mtardy and a team as code owners June 17, 2024 04:08
Copy link

netlify bot commented Jun 17, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 4830be3
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/666fb6b32bd7af00087bc171
😎 Deploy Preview https://deploy-preview-2567--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@mtardy mtardy left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes!

@mtardy mtardy added area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation. labels Jun 18, 2024
@mtardy mtardy merged commit 4e8480f into cilium:main Jun 18, 2024
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants