Skip to content

docs(recipes): add tsheadroom integration guide - #3

Merged
armstrongl merged 2 commits into
mainfrom
larah/tsheadroom
Jun 10, 2026
Merged

docs(recipes): add tsheadroom integration guide#3
armstrongl merged 2 commits into
mainfrom
larah/tsheadroom

Conversation

@armstrongl

@armstrongl armstrongl commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Description

Contributor checklist

  • Copied the integration template from templates/integration.md
  • Placed submission in the correct directory under integrations/
  • Filled in all required frontmatter fields
  • Included working code examples
  • Supplemental files (if any) are referenced from the README
  • Tested the integration against a current Aperture instance

Reviewer checklist

  • Template compliance: all required sections present
  • Technical accuracy: hook definition, grant wiring, and response format snippets are valid and sufficient for adaptation
  • Hook type categorization is correct (integration_type and additional_types if applicable)
  • The dst key requirement for tailnet grants is called out
  • If hook uses modify action on requests: cache impact implications are noted
  • Security review: no credentials, tokens, or sensitive data in examples
  • External links are valid and point to maintained resources
  • CLA signed and commits are DCO signed-off

@armstrongl armstrongl self-assigned this Jun 9, 2026
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@armstrongl
armstrongl requested a review from lkosewsk June 9, 2026 21:38

@lkosewsk lkosewsk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yes! tsheadroom is an excellent pre_request hook example! Thanks for including it!

@armstrongl
armstrongl merged commit e9fc573 into main Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants