Skip to content

Remove outdated UI docs - see main repo instead - #24

Merged
dvail merged 1 commit into
mainfrom
dv/remove-outdated-docs
Jul 8, 2026
Merged

Remove outdated UI docs - see main repo instead#24
dvail merged 1 commit into
mainfrom
dv/remove-outdated-docs

Conversation

@dvail

@dvail dvail commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

All of this information is either duplicated and available in stackrox/stackrox, or worse, horribly outdated.

@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 8c57bfbc-eda1-4472-a2d6-76259c00c372

📥 Commits

Reviewing files that changed from the base of the PR and between d50eb95 and 2838169.

📒 Files selected for processing (6)
  • docs/knowledge-base/[FE] Accessibility.md
  • docs/knowledge-base/[FE] Connecting-your-UI-to-a-large-scale-cluster.md
  • docs/knowledge-base/[FE] How-to-run-vuln-audit-and-analysis-for-UI-dependencies.md
  • docs/knowledge-base/[FE] How-to-update-vulnerable-UI-dependency.md
  • docs/knowledge-base/[FE] Setting-up-multiple-clusters-for-local-FE-development.md
  • docs/knowledge-base/[FE] Use-remote-Central-for-local-front-end-dev.md
💤 Files with no reviewable changes (6)
  • docs/knowledge-base/[FE] How-to-update-vulnerable-UI-dependency.md
  • docs/knowledge-base/[FE] Connecting-your-UI-to-a-large-scale-cluster.md
  • docs/knowledge-base/[FE] Setting-up-multiple-clusters-for-local-FE-development.md
  • docs/knowledge-base/[FE] Use-remote-Central-for-local-front-end-dev.md
  • docs/knowledge-base/[FE] How-to-run-vuln-audit-and-analysis-for-UI-dependencies.md
  • docs/knowledge-base/[FE] Accessibility.md

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Removed several front-end knowledge base pages from the repository, including topics related to accessibility, cluster setup, dependency audits, dependency updates, and local development workflows.

Walkthrough

This pull request removes six frontend knowledge-base documentation files covering accessibility, cluster connectivity, dependency vulnerability auditing, dependency updates, multi-cluster local development setup, and remote Central usage. No code or exported entities are affected.

Changes

Cohort / File(s) Summary
Knowledge-base documentation removal
docs/knowledge-base/[FE] Accessibility.md, docs/knowledge-base/[FE] Connecting-your-UI-to-a-large-scale-cluster.md, docs/knowledge-base/[FE] How-to-run-vuln-audit-and-analysis-for-UI-dependencies.md, docs/knowledge-base/[FE] How-to-update-vulnerable-UI-dependency.md, docs/knowledge-base/[FE] Setting-up-multiple-clusters-for-local-FE-development.md, docs/knowledge-base/[FE] Use-remote-Central-for-local-front-end-dev.md
Deletes entire content of six frontend knowledge-base markdown documentation files.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: removing outdated UI docs and pointing readers to the main repo.
Description check ✅ Passed The description is directly related to the docs removal and the rationale that the content is duplicated or outdated.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dv/remove-outdated-docs

Comment @coderabbitai help to get the list of available commands.

@iamkirkbater

Copy link
Copy Markdown

+1 from me. Always happy to remove old/duplicate docs.

@dvail
dvail merged commit 1306bd3 into main Jul 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants