steelbrain / exec

Node's Process spawning APIs beautified
MIT License
9 stars 4 forks source link

Change default timeout to 10 seconds #76

Closed steelbrain closed 7 years ago

steelbrain commented 7 years ago

Thanks @Arcanemagus for reporting the bug in https://github.com/steelbrain/atom-linter/issues/154

Arcanemagus commented 7 years ago

Are we sure we want to change the default here? Wouldn't it be better to leave this library as a generic library and leave the specific use case such as this timeout in the atom-linter client?

steelbrain commented 7 years ago

I think I agree with you, because this library is going to be used for all sorts of stuff, atom-linter is going to be linter only and while 10 seconds for linters are too much, they might not be for other software