jfmengels / node-elm-review

CLI for elm-review
https://package.elm-lang.org/packages/jfmengels/elm-review/latest/
BSD 3-Clause "New" or "Revised" License
48 stars 25 forks source link

Replace `rimraf` with native `rm` #297

Closed lishaduck closed 3 weeks ago

lishaduck commented 3 weeks ago

We should check if this regresses anything on Windows before merging.

lishaduck commented 3 weeks ago

Also, the test failure looks legit.

socket-security[bot] commented 3 weeks ago

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/elm-doc-preview@5.0.5 filesystem Transitive: environment, eval, network, shell, unsafe +50 4.37 MB rlefevre
npm/npm-run-all@4.1.5 environment Transitive: filesystem +31 700 kB mysticatea

View full report↗︎

jfmengels commented 3 weeks ago

Thanks! One less dependency!