Sindre Sorhus comments

Results 4160 comments of


                                            Sindre Sorhus

use system binary if possible

I don't think that's a good idea. The system binary might be outdated and missing CLI flags that the user has supplied or differing behavior.

Use binary destination set in package.json

ping @kevva

Use binary destination set in package.json

Yes, but you have to delete the cache. See https://github.com/sindresorhus/meow/blob/d055091dff7a9dc9a75bc619852a4ffacda2d6fb/index.js#L8-L10

I wish I didn't have to create this boilerplate file: https://github.com/flowtype/flow-bin/blob/master/lib/install.js Also wish I could define everything in one file instead of having to have multiple: `lib/index.js` `lib/install.js` `index.js` `cli.js`....

Simplify bin creation

Yup.

Warn users of this lib (add to docs) that all binaries must be statically compiled

:+1:

process without vscode

Please elaborate. Not clear what you’re saying.

Cleanup script shouldn't delete config files when updating workflow

What configuration files? Also see: https://github.com/sindresorhus/alfy/issues/36

Maintain script through package.json

👍 `bin`

Maintain script through package.json

Actually, one problem with using `bin` is that npm will add it as a global binary when the user install the workflow, so while `bin` is more correct semantically, `main`...