Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Clean Up Dependency Injection #255

Open
CianLR opened this issue Dec 1, 2018 · 0 comments
Open

Clean Up Dependency Injection #255

CianLR opened this issue Dec 1, 2018 · 0 comments

Comments

@CianLR
Copy link
Member

CianLR commented Dec 1, 2018

@devoxel mentioned it's a bit awkward the amount of DI going on, logger having to be injected into everything for one. There may be a nicer way to handle these things, one of the suggestions was to use class variables instead which might work.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

1 participant