search

jaredhanson / electrolyte

Elegant dependency injection for Node.js.
MIT License
564 stars 61 forks source link

add auto resolver #73

Open piercus opened 6 years ago

piercus commented 6 years ago

Hello @jaredhanson

I've seen in the code that @implements and @requires are defined.

I think it makes sense to add https://github.com/bixbyjs/bixby/blob/master/lib/resolvers/auto.js into electrolyte and to include it by default.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-2.8%) to 90.69% when pulling 7929a2d973b5a81f79903c429010e9ad71e25a75 on piercus:auto_resolver into d6af75b444e8c46bd62c2eb2b708b73af3145e62 on jaredhanson:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-2.8%) to 90.69% when pulling ac7a0325e66d41780fa4e3becc7cf9e0a8d6528d on piercus:auto_resolver into d6af75b444e8c46bd62c2eb2b708b73af3145e62 on jaredhanson:master.