breaking: use parseArgs from Node 18.3+ instead of sade/mri#2814
breaking: use parseArgs from Node 18.3+ instead of sade/mri#2814aewing wants to merge 5 commits intosveltejs:masterfrom
Conversation
- Replace sade dependency with Node.js built-in parseArgs from node:util - Update engines requirement to Node.js >= 18.3.0 (when parseArgs was added) - Remove sade and @types/sade dependencies - Implement custom help and version handling to match original behavior - Add proper TypeScript types for parseArgs options Resolves sveltejs#2521
|
@benmccann @jasonlyu123 let me know if you had something else in mind, happy to address any feedback. |
|
Looks good, but we should only merge this once we cut a new svelte-check release. Putting it into draft mode to prevent accidental merges. Thank you! |
|
|
@dummdidumm any thoughts on this one now? worth keeping/merging, or any rework or reconsideration you'd like here? |
Resolves #2521