Adding #42: Add options.array

[ianwalter]

#42

[codecov-io]

Codecov Report

Merging #43 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #43   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          94     95    +1     
=====================================
+ Hits           94     95    +1

Impacted Files	Coverage Δ
index.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fcb1d61...4b82420. Read the comment docs.

[jorgebucaran]

Great work, @ianwalter! I'm going to run this through the benchmark before I make a decision.

Thanks!

[jorgebucaran]

@ianwalter Mind sharing your benchmark results? :)

[ianwalter]

getopts × 1,348,413 ops/sec
minimist × 274,594 ops/sec
yargs × 34,735 ops/sec
mri × 719,357 ops/sec

On

MacBook Pro (15-inch, 2017)
2.8 GHz Intel Core i7
16 GB 2133 MHz LPDDR3

[jorgebucaran]

@ianwalter Do you think you can update getopts.d.ts too?

[ianwalter]

@ianwalter Do you think you can update getopts.d.ts too?

Sorry, was on vacation and forgot about this. Will update it now.

[ndabAP]

Hey @jorgebucaran, could we merge this?

[jorgebucaran]

@ndabAP Docs are still missing (I did forget to request them, though). Let me see if I can manage to write them this week or over the weekend and merge this once and for all! 💯

[jorgebucaran]

@ianwalter @ndabAP To clarify, this feature lets you specify which options must be treated as arrays, but we're still making arrays from repeated options anyway. The difference is you'd need to check if that option was an array or not yourself before trying to access it. Is this the desired behavior?