MAESTRO: Close PR #882 in PR-Triage-10 - generic Windows notes duplicated by existing docs
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -35,7 +35,8 @@ Quick wins — docs PRs are low-risk and can be merged rapidly after basic revie
|
||||
- [x] Review and merge PR #919 (`docs: add Windows setup note for npm not recognized error` by @kamran-khalid-v9). Steps: (1) `gh pr checkout 919` (2) Review the note — should explain PATH configuration for npm on Windows (3) If helpful: `gh pr merge 919 --rebase --delete-branch`
|
||||
- **Merged 2026-02-06.** Added 11-line Windows setup note to README.md explaining how to resolve "npm is not recognized as the name of a cmdlet" error by installing Node.js and ensuring npm is on PATH. CI passed (Greptile Review). Fixes #908.
|
||||
|
||||
- [ ] Review and merge PR #882 (`Add Windows local development notes to README` by @namratab18). Single file. Steps: (1) `gh pr checkout 882` (2) Quick review (3) `gh pr merge 882 --rebase --delete-branch`
|
||||
- [x] Review and merge PR #882 (`Add Windows local development notes to README` by @namratab18). Single file. Steps: (1) `gh pr checkout 882` (2) Quick review (3) `gh pr merge 882 --rebase --delete-branch`
|
||||
- **Closed 2026-02-06 (not merged).** Content was too generic — the 3 steps (install Git, clone repo, run project) aren't Windows-specific and duplicate the existing Development Guide link. README already has a "Windows Setup Notes" section (from PR #919) covering the most common Windows issue (npm PATH). Greptile review also flagged awkward placement and vague instructions.
|
||||
|
||||
## Issue Templates
|
||||
|
||||
|
||||
Reference in New Issue
Block a user