You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Claude finished @github-actions[bot]'s task in 3m 26s —— View job
Review: CLI and Helm reference update for v2.36.2
Gather context and read CLAUDE.md
Read doc-review skill
Review changed files for docs conventions
Check for relative links, missing navigation entries, and content issues
Post findings
Reviewed all 23 changed files. This is a well-structured automated update — no new pages, no relative links, no missing navigation entries. The changes break down into four categories:
1. Scaling flags removed from CLI (kosli create environment)
The --include-scaling and --exclude-scaling flags and their explanatory paragraph are removed. This is consistent with the Helm chart deprecation (see below) and appears to be a clean removal of a feature that no longer has any effect.
2. New CI guidance for delete commands
kosli_delete_api-key.md and kosli_delete_service-account.md gain a paragraph explaining that non-interactive stdin (e.g. CI) requires --assume-yes. Clear and useful addition.
3. Helm chart updates (chart v2.6.0 → v2.7.0)
overview.mdx — Version bumped to chart 2.7.0 / CLI v2.36.1.
configuration.mdx — reporterConfig.includeScaling description updated to document it as deprecated and ignored.
upgrading.mdx — New migration note for v2.7.0 explaining the includeScaling deprecation. Well-written, includes the "what changed" and "what to do" clearly.
4. Live example output refreshes
The bulk of the diff (+2060/−2331) is updated JSON example output in kosli_get_snapshot.md, kosli_diff_snapshots.md, kosli_log_environment.md, kosli_list_environments.md, and kosli_list_snapshots.md. These are snapshot-in-time data from the cyber-dojo demo org and are expected to drift with each release.
Suggestions (non-blocking):
helm/k8s_reporter/overview.mdx:7 — The appVersion reference says v2.36.1 but this PR is for v2.36.2. If the chart's appVersion was actually bumped to v2.36.2, this line should say so. If the chart still pins to v2.36.1, then this is correct as-is. Worth a quick check against the actual chart metadata.
No critical or improvement-level issues found. All internal links use root-relative paths, frontmatter is intact, no new pages were added without navigation entries, and writing style is consistent with conventions.
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
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.
Automated update of CLI and Helm reference documentation for release
v2.36.2.