search

minimistjs / minimist

parse argument options
MIT License
515 stars 30 forks source link

Fixed Issue #44 #45

Closed cantwellsean0127 closed 11 months ago

cantwellsean0127 commented 11 months ago

[v1.2.8] Set default args parameter value to process.argv.split(2)

codecov-commenter commented 11 months ago

Codecov Report

Merging #45 (0d3cf8e) into main (fdbb909) will not change coverage. The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage   98.78%   98.78%           
=======================================
  Files           1        1           
  Lines         165      165           
  Branches       70       71    +1     
=======================================
  Hits          163      163           
  Misses          2        2
Files Changed Coverage Δ
index.js 98.78% <100.00%> (ø)
shadowspawn commented 11 months ago

(I started the tests, but of course they won/t show up any problems since this won't be covered by the existing tests.)

This Code does not like it will work. Have you tried it?

You can mark a Pull Request as Draft while you are still working on it and especially when not ready for review.

ljharb commented 11 months ago

→ #46.