sindresorhus / globby

User-friendly glob matching
MIT License
2.52k stars 130 forks source link

Update index.js #165

Closed vladimir-ivanov closed 3 years ago

vladimir-ivanov commented 3 years ago

fails lower versions of nodejs - e.g. we use node 8

vladimir-ivanov commented 3 years ago

please merge ASAP and do a new npm release if possible? breaks a lot of libraries for us, upgrading nodejs is not an option, or instead mark it as a breaking change? https://github.com/jhotmann/node-rename-cli/blob/master/package.json#L30 breaks in our case many thanks

sindresorhus commented 3 years ago

This package targets Node.js 10: https://github.com/sindresorhus/globby/blob/4bb7f4a3ffb1a4ef23c86fbf70261110513e1fcc/package.json#L14

I'm not interested in this change.

sindresorhus commented 3 years ago

instead mark it as a breaking change?

It was: https://github.com/sindresorhus/globby/releases/tag/v11.0.0