Filing because I want to make sure it gets seen. From https://github.com/jashkenas/coffeescript/pull/4697/files#r139525428
2.x has [email protected] as an optionalDependency, but since those are auto-installed, and there's never a time where babel-core should fail to install, it's essentially been made a direct dependency.
Since the logic to load it is already behind atry/catch with a notice to install it, it seems like the optionalDependencies block should just be removed. This also opens up users to the possibility of using Babel 7.x in the future, since that will be released in the coming months. As it is now, it'll be a breaking change for coffeescript@2 to change it to [email protected] which would be extremely unfortunate.
Filing because I want to make sure it gets seen. From https://github.com/jashkenas/coffeescript/pull/4697/files#r139525428
2.xhas[email protected]as anoptionalDependency, but since those are auto-installed, and there's never a time wherebabel-coreshould fail to install, it's essentially been made a direct dependency.Since the logic to load it is already behind a
try/catchwith a notice to install it, it seems like theoptionalDependenciesblock should just be removed. This also opens up users to the possibility of using Babel 7.x in the future, since that will be released in the coming months. As it is now, it'll be a breaking change forcoffeescript@2to change it to[email protected]which would be extremely unfortunate.