Skip to content

refactor: replace rimraf with built-in fs.rmSync#109

Draft
Garbee wants to merge 1 commit intogarbee/remove/sinonfrom
garbee/remove/rimraf
Draft

refactor: replace rimraf with built-in fs.rmSync#109
Garbee wants to merge 1 commit intogarbee/remove/sinonfrom
garbee/remove/rimraf

Conversation

@Garbee
Copy link
Copy Markdown
Member

@Garbee Garbee commented Mar 20, 2026

Summary

  • Replaces rimraf dev dependency with Node.js built-in fs.rmSync in the prebuild script
  • Continues the pattern of removing third-party dependencies in favor of native alternatives

Test plan

  • yarn prebuild succeeds when dist exists (removes it)
  • yarn prebuild succeeds when dist doesn't exist (no error)
  • Pre-commit hook rebuilds dist successfully

No QA Required

@Garbee Garbee self-assigned this Mar 20, 2026
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.

1 participant