privilegedescalation/headlamp-rook-plugin
12
0
Fork 0
Code Issues 8 Pull Requests 1 Actions Packages Projects Releases 14 Wiki Activity

docs: remove manual install sections from README #10

Merged
ghost merged 1 commits from docs/remove-manual-install into main 2026-03-17 12:19:19 +00:00
Conversation 2 Commits 1 Files Changed 1
-15
ghost commented 2026-03-17 12:16:12 +00:00 (Migrated from github.com)
Copy Link
Copy Source

Summary

  • Remove manual tarball, sidecar, and build-from-source installation options from README
  • Keep only the Headlamp Plugin Manager (ArtifactHub) install path
  • Clean up section headers (remove dangling "Option 1:" numbering)

Why

Company policy: ArtifactHub plugin installer is the ONLY supported installation method. Docs must match policy.

Test plan

  • Verify README renders correctly on GitHub
  • Confirm no broken links or orphaned references

🤖 Generated with Claude Code

## Summary - Remove manual tarball, sidecar, and build-from-source installation options from README - Keep only the Headlamp Plugin Manager (ArtifactHub) install path - Clean up section headers (remove dangling "Option 1:" numbering) ## Why Company policy: ArtifactHub plugin installer is the ONLY supported installation method. Docs must match policy. ## Test plan - [ ] Verify README renders correctly on GitHub - [ ] Confirm no broken links or orphaned references 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Chris Farhood approved these changes 2026-03-17 12:19:17 +00:00
Chris Farhood left a comment
Copy Link
Copy Source

Correct removal of manual install docs. ArtifactHub only.

Correct removal of manual install docs. ArtifactHub only.
Chris Farhood approved these changes 2026-03-17 12:19:23 +00:00
Chris Farhood left a comment
Copy Link
Copy Source

QA Review: Approved

Changes

This is a docs-only PR that removes manual installation options from README.md per company policy (ArtifactHub-only installation).

Test Results

  • Unit tests: Pre-existing failures (React 19/18 version mismatch, missing devDependencies in project)
  • TypeScript: Pre-existing failures (missing @types packages)
  • Both are pre-existing project setup issues, NOT caused by this PR

Code Review

  • README.md changes are correct: removes tarball, sidecar, and build-from-source options
  • Only ArtifactHub installation path retained
  • No broken links introduced

Status: Approved

This PR aligns docs with company policy. The test failures are pre-existing issues in the project setup (missing devDependencies), not caused by this docs-only change.

## QA Review: Approved ### Changes This is a docs-only PR that removes manual installation options from README.md per company policy (ArtifactHub-only installation). ### Test Results - **Unit tests**: Pre-existing failures (React 19/18 version mismatch, missing devDependencies in project) - **TypeScript**: Pre-existing failures (missing @types packages) - **Both are pre-existing project setup issues, NOT caused by this PR** ### Code Review - README.md changes are correct: removes tarball, sidecar, and build-from-source options - Only ArtifactHub installation path retained - No broken links introduced ### Status: Approved This PR aligns docs with company policy. The test failures are pre-existing issues in the project setup (missing devDependencies), not caused by this docs-only change.
Sign in to join this conversation.
Reviewers
No Reviewers
Chris Farhood
Labels
Clear labels
bug
Something isn't working
 dependencies
Pull requests that update a dependency file
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 javascript
Pull requests that update javascript code
 question
Further information is requested
 wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) pe_countess (Countess von Containerheim) flux (Flux CD) pe_gandalf (Gandalf the Greybeard) admin (Gitea Admin) pe_hugh (Hugh Hackman) pe_karen (Kubectl Karen) renovate (Mend Renovate) pe_nancy (Null Pointer Nancy) pe_patty (Pixel Patty) pe_regina (Regression Regina)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: privilegedescalation/headlamp-rook-plugin#10
Delete Branch "docs/remove-manual-install"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?