search

Escape-Technologies / mookme

A pre-commit tool designed for monorepos.
MIT License
102 stars 13 forks source link

'rm' doesn't exist on Windows #84

Closed andrewalderson closed 2 years ago

andrewalderson commented 2 years ago

When working on a fix for #83 I was getting an error with the npm prebuild script because the rm utility doesn't exist on Windows. There is a cross platform package (rimraf) that can be used if you want people using Windows to submit PRs.

Solution:

npm i -D rimraf and change the npm prebuild script to "prebuild": "rimraf -f ./.tmp/ts-build-infos.json && rimraf -rf dist"

LMaxence commented 2 years ago

I actually do want that :) You can add that dependency as a dev dependency at the root of the monorepo

LMaxence commented 2 years ago

Sorry for closing this, I missclicked