yuuko
yuuko copied to clipboard
Stop exporting Eris namespace from yuuko
Makes docs messy, re-exports make it unclear which package provides what, and one more require('eris')/import 'eris' isn't the end of the world for anyone
I think it's A-ok to re-export
I agree it's okay, but I don't actually know if it's that helpful, and it adds clutter to the Yuuko API docs. In order to use the re-export, you have to do e.g. const {Eris} = require('yuuko'); and then reference things as e.g. Eris.Collection, whereas if you add a second import for Eris directly you can just do const {Collection} = require('eris'); and use Collection directly.
Ultimately I think the cases for having to use Eris classes directly are pretty uncommon unless you're using Typescript, and TS folks should be more used to having more imports at the top of the file anyway.
All that said, it's gonna be a while before I do anything about this either way - this will have to be in Yuuko v3, which is a long way away still.
Should do a deprecation for this before v3