Updates to the docs#4
Merged
javisperez merged 3 commits intomainfrom Mar 25, 2026
Merged
Conversation
Signed-off-by: Javis Pérez <javis@jozu.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the kitops-ts documentation to improve clarity and reduce duplication, including moving type docs into a dedicated page and refining examples and navigation.
Changes:
- Added a new
docs/types.mdpage to document exported TypeScript types (and removed redundant type docs fromdocs/api-reference.md). - Updated
docs/examples.mdwith a more “production-like” workflow (Kitfile generation, env-based config, andlogoutinfinally). - Cleaned up docs navigation/content and adjusted docs sync workflow pathing.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
docs/types.md |
New consolidated type documentation page (needs link/anchor and type-shape accuracy fixes). |
docs/index.md |
Removes badges and contributing link; docs nav text now needs to reflect the new Types page. |
docs/getting-started.md |
Changes KitOps CLI installation link to a relative site path. |
docs/examples.md |
More robust example, but currently can break when env vars/workdir are missing. |
docs/api-reference.md |
Removes redundant Types section; kit() example now needs to match actual function signature. |
.github/workflows/docs-sync.yml |
Updates destination docs path for sync into the kitops repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Javis Pérez <javis@jozu.com>
Signed-off-by: Javis Pérez <javis@jozu.com>
amisevsk
approved these changes
Mar 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the documentation for the
kitops-tsTypeScript SDK, focusing on improving clarity, usability, and accuracy. The main changes include updating example code to be more robust, correcting documentation paths, and cleaning up outdated or redundant content.Documentation improvements:
docs/examples.mdto generate aKitfileonly if it doesn't exist, use environment variables for configuration, and handle login/logout more robustly. This makes the example more production-ready and user-friendly. [1] [2]docs/getting-started.mdto use a relative path, ensuring the link works correctly in the documentation site.docs/index.mdby removing badges and an outdated contributing link, resulting in a more focused introduction. [1] [2]API reference and workflow updates:
docs/api-reference.md, likely to reduce duplication and keep the API reference concise.KITOPS_TS_DOCS_PATHin.github/workflows/docs-sync.ymlto reflect the new documentation directory structure, ensuring CI/CD workflows point to the correct location.