unjs / nypm

🌈 Unified Package Manager for Node.js and Bun
MIT License
446 stars 15 forks source link

feat: `addDevDependacy`, `removeDevDependacy`, `removeDependacy` #31

Closed cpreston321 closed 1 year ago

cpreston321 commented 1 year ago

Resolves: #5

Added new functions based on Issue. Added more test.

codecov[bot] commented 1 year ago

Codecov Report

Merging #31 (cb1bbfe) into main (02e9a14) will increase coverage by 1.27%. The diff coverage is 100.00%.

:exclamation: Current head cb1bbfe differs from pull request most recent head ae53fb8. Consider uploading reports for the commit ae53fb8 to get more accurate results

@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   97.32%   98.59%   +1.27%     
==========================================
  Files           5        6       +1     
  Lines         112      214     +102     
  Branches       28       37       +9     
==========================================
+ Hits          109      211     +102     
  Misses          3        3              
Impacted Files Coverage Δ
src/_utils.ts 100.00% <100.00%> (ø)
src/api.ts 100.00% <100.00%> (ø)
src/detect.ts 97.43% <100.00%> (-0.72%) :arrow_down:
src/spawn.ts 90.90% <100.00%> (-1.40%) :arrow_down:
src/types.ts 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

pi0 commented 1 year ago

Reworking in #55 to resolve merge conflicts. Thanks again for nice work on this ❤️