stdin support

[sonnyp]

fixes #4

[SamVerschueren]

Missing

if (!input && process.stdin.isTTY) {
    console.error('Input required');
    process.exit(1);
}

More info as to why this should be added

• stdin handling meow#20
• Example: https://github.com/sindresorhus/find-versions-cli/blob/56dbd77051147819abdf845504af3b3cc1dd2935/cli.js#L30-L41

[sonnyp]

cc @SamVerschueren

[SamVerschueren]

Why not just do it like this?

const input = cli.input[0];

if (!input && process.stdin.isTTY) {
    console.error('Input required');
    process.exit(1);
}

if (input) {
    opn(input, cli.flags);
} else {
    // stdin stuff here
}

You have to show an error if input is empty and process.stdin.isTTY is true.

[SamVerschueren]

Oh you are showing an error. Would rather see it extracted from the this if-else block because this looks weird.

1. handle input
2. show error
3. handle input

[sonnyp]

done

[SamVerschueren]

Thanks @sonnyp ! Looks good to me.

[sonnyp]

@sindresorhus r?

[corysimmons]

@sonnyp 💯 :D

[sindresorhus]

Thank you @sonnyp :)