Skip to content

Conversation

MickaelFontes
Copy link
Contributor

Fixes some reference links on Hook points page

Description

Some links in the documentation are broken.

@MickaelFontes MickaelFontes requested review from mtardy and a team as code owners April 15, 2025 19:33
Copy link

netlify bot commented Apr 15, 2025

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 9b3c9e3
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/6813976b718d16000830558c
😎 Deploy Preview https://deploy-preview-3642--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.

Thank you!

@mtardy mtardy added area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation. labels Apr 16, 2025
@MickaelFontes
Copy link
Contributor Author

Thanks for the corrected suggestion! I also noticed some broken links in the pull request template, I'll rename the pull request and add them here also.

@MickaelFontes MickaelFontes changed the title doc: fix reference links on Hook points page doc: fix some links in documentation Apr 30, 2025
@mtardy
Copy link
Member

mtardy commented Apr 30, 2025

Thanks for the corrected suggestion! I also noticed some broken links in the pull request template, I'll rename the pull request and add them here also.

Thank you, awesome, could you just squash the two first commits? Making:

@MickaelFontes
Copy link
Contributor Author

Thanks for the corrected suggestion! I also noticed some broken links in the pull request template, I'll rename the pull request and add them here also.

Thank you, awesome, could you just squash the two first commits? Making:

Sure, it should be good now!

@ghost
Copy link

ghost commented May 1, 2025

@MickaelFontes Sorry, the commit check is unhappy because the first commit now has duplicated "Signed-off-by" line and capitalized "Co-Authored-By" (should be "Co-authored-by").

https://github.com/cilium/tetragon/actions/runs/14759617206/job/41481390390?pr=3642

Could you amend the commit message and force push? Then it will be all good.

MickaelFontes and others added 2 commits May 1, 2025 17:46
doc: use tagged version instead of main

Signed-off-by: MickaelFontes <[email protected]>
Co-authored-by: Mahé <[email protected]>
@mtardy
Copy link
Member

mtardy commented May 15, 2025

Thank you!!

@mtardy mtardy merged commit c471a5f into cilium:main May 15, 2025
26 of 28 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.

3 participants