Reduce lib size

[icebob]

Hi,

I checked your lib size because it is~300kB: https://arve0.github.io/npm-download-size/#args
But your code is very small (1kB), but you have "string-similarity" dependency which is 300kB. 86% of the whole size.

Do you plan to change it to other smaller lib?

Thanks!

[ntwcklng]

Hey @icebob,

args is a CLI which only runs on Desktops, so in my opinion the size does not matter that much as on mobile web apps. But in general i like smaller and lighter modules.
Do you know a smaller module like string-similarity?
string-similarity depends on several lodash functions (=99% of the size), so maybe we can create a PR without lodash.

[icebob]

I found leven. It's only 2.3kB, but it's a Levenshtein distance algorithm.

[ntwcklng]

Okay, do you want to create a PR?

[icebob]

Yes, I will.

[ntwcklng]

Thanks! Maybe we should wait until #116 is merged?!

[icebob]

When will it be merged?

[ntwcklng]

Done 👍

[icebob]

Great! I'm trying to change similarity lib...

[icebob]

I'm trying, but how can I test it? I see there is no test case for similarity and I don't know when this code is called. Could you help me?

My changes

[ntwcklng]

You can create a file, where you require your local args module like this:

similarity.js

#!/usr/bin/env node
const args = require('../args/lib/')

args
    .option('t', 'This is a test', 'foo')
    .option('verbose', 'verbose option', 'jep')

const flags = args.parse(process.argv, {
  version: false,
  mri: {
    string: 't'
  }
})
console.log(flags)

switch to that dir and run node similarity.js and test some typos: node similarity.js --ferbose, node similarity.js --vernose

[icebob]

I'm done. Could you check it? #121

[icebob]

@ntwcklng When can you check & merge it?

[ntwcklng]

I will look at it next week 👍

[ntwcklng]

Released in 4.0.0, thanks @icebob 😎

[icebob]

Great! Me too! 😎