Skip to content

docs(skyhook): add integration documentation about skyhook#453

Merged
mchmarny merged 1 commit intomainfrom
docs/skyhook
Mar 21, 2026
Merged

docs(skyhook): add integration documentation about skyhook#453
mchmarny merged 1 commit intomainfrom
docs/skyhook

Conversation

@ayuskauskas
Copy link
Copy Markdown
Contributor

Summary

Documentation about Skyhook and skyhook-customizations

Motivation / Context

Provide more information for integrators

Fixes:
Related:

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)
  • Build/CI/tooling

Component(s) Affected

  • CLI (cmd/aicr, pkg/cli)
  • API server (cmd/aicrd, pkg/api, pkg/server)
  • Recipe engine / data (pkg/recipe)
  • Bundlers (pkg/bundler, pkg/component/*)
  • Collectors / snapshotter (pkg/collector, pkg/snapshotter)
  • Validator (pkg/validator)
  • Core libraries (pkg/errors, pkg/k8s)
  • Docs/examples (docs/, examples/)
  • Other: ____________

Risk Assessment

  • Low — Isolated change, well-tested, easy to revert
  • Medium — Touches multiple components or has broader impact
  • High — Breaking change, affects critical paths, or complex rollout

Rollout notes:

Checklist

  • Tests pass locally (make test with -race)
  • Linter passes (make lint)
  • I did not skip/disable tests to make CI green
  • I added/updated tests for new functionality
  • I updated docs if user-facing behavior changed
  • Changes follow existing patterns in the codebase
  • Commits are cryptographically signed (git commit -S) — GPG signing info

@ayuskauskas ayuskauskas requested a review from a team as a code owner March 21, 2026 01:13
@github-actions
Copy link
Copy Markdown

Welcome to AICR, @ayuskauskas! Thanks for your first pull request.

Before review, please ensure:

  • All commits are signed off per the DCO
  • CI checks pass (tests, lint, security scan)
  • The PR description explains the why behind your changes

A maintainer will review this soon.

@mchmarny mchmarny merged commit 53892a9 into main Mar 21, 2026
12 checks passed
@mchmarny mchmarny deleted the docs/skyhook branch March 21, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants