Repository navigation
docs: repository - Remove obsolete guidance and documents - #481
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for thisonetimeonacid ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
PR Template Routing 📋Branch prefix: docs Suggested template: pr_documentation.md Type: Documentation PR Canonical templates are managed in lightspeedwp/.github:
|
Pre-merge checks:
 |
| Check name | Status | Explanation |
|---|---|---|
| Description Check |  |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
| Title check | |
The title clearly identifies a documentation cleanup that removes obsolete guidance and documents. It is concise and related to the main changes, although it does not mention the replacement of approv… |
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (5 skipped: 5 unsupported.)
Full details: Title check
-
Explanation
The title clearly identifies a documentation cleanup that removes obsolete guidance and documents. It is concise and related to the main changes, although it does not mention the replacement of approved documents.
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
Drift Detection ✅✅ No drift detected. Configuration is canonical. Canonical governance files are correctly centralized in lightspeedwp/.github. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Queued — the merge queue status continues in this comment ↓. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🔇 Additional comments: 5
docs/INTEGRATION-VALIDATION-PLAN.md: 1
1-20: LGTM!Also applies to: 22-41, 42-56, 57-72, 73-92, 94-124, 126-145, 147-173, 175-201, 203-213
docs/dark-mode-troubleshooting.md: 1
2-126: LGTM!docs/ebook-preferences.md: 1
1-70: LGTM!docs/NETLIFY_CACHE_VERIFICATION.md: 1
164-167: 📐 Maintainability & Code QualityThe patch-evidence links are accessible.
Both commit URLs return HTTP 200 without authentication. No link replacement or qualification is required.
docs/TESTING.md-145-146: 1
145-146:Minor · Functional Correctness · Quick win
⚠️ Unverified finding
Verification ran but could not confirm this finding. It is shown for review, not as a verified issue.Qualify the
postinstallclaim.
npm ciruns thepostinstallscript, but that script exits beforelefthook installwhen the command is outside a Git worktree. State that Lefthook is installed only inside a Git worktree.
-
-
- Source mode: Auto
- Repository: lightspeedwp/ThisOneTimeOnAcidCom
- Branch: docs/repository-legacy-cleanup
- File: .coderabbit.yml
- Commit: 5607ce8
- Layers: Repository YAML (base), Central YAML (inherited)
- Review profile: CHILL
- Plan: Essentials
- Run ID:
48b2bfb6-6747-4842-839a-11d189c23c68
-
docs/INTEGRATION-VALIDATION-PLAN.mddocs/NETLIFY_CACHE_VERIFICATION.mddocs/TESTING.mddocs/dark-mode-troubleshooting.mddocs/ebook-preferences.mddocs/figma-ai-brief-1.mddocs/makeup-kit-essentials.mddocs/pending-questions.md
-
- docs/figma-ai-brief-1.md
- docs/makeup-kit-essentials.md
- docs/pending-questions.md
Included review availability: This review used your included allowance. 1 included review remains after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.
-
ashleyshaw
changed the title
|
@Mergifyio rebase |
☑️ Nothing to do, the required conditions are not metDetails
|
|
@Mergifyio queue |
Merge Queue Status
This pull request spent 17 seconds in the queue, including 1 second running CI. Required conditions to merge |
Documentation Pull Request
Linked issues
Relates to #448 (TOA-366). This is a bounded contribution; the wider documentation outcome remains open. No parent epic or closing target is proposed.
What changed
docs/ebook-preferences.md,docs/TESTING.md,docs/dark-mode-troubleshooting.md,docs/NETLIFY_CACHE_VERIFICATION.mdanddocs/INTEGRATION-VALIDATION-PLAN.mdwith the exact owner-approved uploads. The updated testing attachment retains the original review provenance, adds the native-test row naming both files, and describes unit → native → build → E2E.docs/figma-ai-brief-1.mdand obsoletedocs/makeup-kit-essentials.md; preserve the canonical brief.docs/pending-questions.md; preserve every other byte, including image1 and image2.Audience & placement
docs/.Preview / Screenshots
Documentation-only diff; no UI screenshots. No runtime, storage, stylesheet, test, dependency, hosting, header or workflow changes. The historical PR-426 validation record is unchanged.
Notes
develop,f99275b085cffd209b9bd42f6e463c0af643099a.3f583002482d80ffd1910676bcf7c63e13989b20. Intervening Sentry changes were inspected; the owner approved the testing command-sequence amendment. Other inspected contracts still support the drafts.lightspeedwp/.github/.github/PULL_REQUEST_TEMPLATE/pr_docs.md; owner-selected type label:type:docs. The existing local router suggestspr_documentation.mdand attemptstype:documentation; no routing or governance repair is included.5607ce8077f8cbb5541fc116c69b8b14d065139f(Node 24.19.0, npm 11.9.0).git fetch origin develop: exit 0.npm run validate:branch-name -- --branch docs/repository-legacy-cleanup: exit 0 before creation.npm run validate:branch-name: exit 0 after creation.python /tmp/toa-366/verify.py: exit 0; all five uploads match byte-for-byte, exactly eight paths changed, replacement fences balanced, inline local link targets exist, precise removal verified, retained material unchanged, and no tracked deletion references found.npm run build: exit 1 because the existing installation lacked@sentry/vite-plugin.npm ci: exit 0; manifest and lockfile unchanged; postinstall installed Lefthook hooks.npm run build: exit 0; chunk-size warning retained.git commit: exit 0; pre-commit branch validation passed; journal validation skipped because no journal files changed.git push -u origin docs/repository-legacy-cleanup: exit 0; pre-push destination-branch validation and build both passed. No hooks bypassed.git diff --check: exit 2; the pre-existing blank line following the removed image3 definition becomes a trailing blank line. It remains intact to preserve the exact authorised deletion scope.5607ce8077f8cbb5541fc116c69b8b14d065139fshow successful relevant checks and preview statuses, with other checks skipped or neutral and cancelled superseded runs. Jest was skipped for this documentation-only contribution. No production deployment, publication or whole-issue acceptance is verified by this update.Changelog
Issue #448 has
meta:no-changelog; CONTRIBUTING.md explicitly exempts it. The protected root changelog is unchanged; no workaround labels are requested.Added
No runtime features.
Changed
Five approved documentation replacements; precise pending-questions cleanup.
Fixed
Source-contract and historical-evidence descriptions in the bounded documents.
Removed
Two approved documents and the unrelated embedded fragment/image3 only.
Checklist (Global DoD / PR)
Summary by CodeRabbit