docs: comprehensive documentation cleanup and alignment#154
Merged
Conversation
README.md: - Add cloudflared chart (was missing entirely!) - Fix chart name: olm (was incorrectly called 'holmes') - Update example commands to use correct chart names - Remove static version numbers (use helm search instead) CLAUDE.md: - Fix versioning section: git-cliff + version-bump.sh (not release-please) - Add Chart Release Pipeline overview table (W1, W2, W5, W6) - Update ADR references to accurate/current ADRs - Fix labels strategy path reference SUMMARY.md: - Add missing ADR-008 and ADR-009 links - Add commit-conventions and atomization contributing guides - Mark ADR-006 as superseded in navigation .claude/plans/: - Archive completed chart-release-workflows plans - Archive cloudflared MVP plan (completed) - Archive helm-workflow-gaps (led to chart-release-workflows) - Rename foo.md → chart-release-workflows-notes.md - Keep future cloudflared enhancement plans (02-07) active Closes #149, Closes #150, Closes #151, Closes #153
Contributor
Atomic Branching PreviewAll extractions verified successfully Branches to be created
Files with no matching pattern (DLQ)The following files don't match any branch pattern and will go to the DLQ:
File Detailsrepo/config
|
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
1 participant
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.
Summary
Major documentation cleanup to align documentation with the actual state of the repository.
Changes
README.md
olmwas incorrectly calledholmeshelm searchinstead).claude/CLAUDE.md
docs/src/SUMMARY.md
.claude/plans/
archive/foo.md→chart-release-workflows-notes.md(it has useful context)Issues Closed
Closes #149 - README.md outdated
Closes #150 - CLAUDE.md stale references
Closes #151 - ADR index inconsistencies
Closes #153 - .claude/plans cleanup
Not Addressed in This PR
Test Plan
charts/directory🤖 Generated with Claude Code