Fix/documentation md#2178
Conversation
|
@Naveen-Boddepalli is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
This PR has merge conflicts with git fetch origin
git rebase origin/main |
695bfe4 to
5c313e0
Compare
|
|
Hi! This PR has merge conflicts with the git fetch upstream
git rebase upstream/main
# resolve any conflicts
git push --force-with-leaseIf you're no longer working on this, let us know and we can close it. Thanks! |
|
resolved conflicts ✅ |
|
The project CI uses |
Summary
Fixed the reported bug plus a second one hiding in plain sight. All changes made:
Reported fix — Node version (line 11)
Node.js >= 18 → Node.js >= 20 to match package.json engines field
Bonus fix — npm → pnpm (6 locations)
package.json explicitly sets "packageManager": "pnpm@9.0.0", but DEVELOPMENT.md told contributors to use npm everywhere. This would cause pnpm's lockfile (pnpm-lock.yaml) to be ignored, potentially installing wrong dependency versions or breaking the workspace setup. Locations fixed:
Closes #2173
Type of Change
Changes Made
fixed the documentation of documentation.md file
How to Test
the fixed documentation will have correct versions.
Screenshots (if UI change)
N/A
Checklist
npm run lintpasses locallynpm run type-check)Accessibility Checklist
Additional Notes
This update standardizes pull request submissions and helps maintain consistent review quality across contributions.