dependency cycle not best practice

[AidenFaulconer]

In this file we can see we require logger ('./../config/logger') which is a workaround from getting a dependency cycle warning, since logger is required in index.js as well (the entry point). Thus forming a dependency cycle.

Is this best practice? Would this cause an incomplete import from happening if this boilerplate was expanded upon in some instances?

[danielfsousa]

We should avoid dependency cycles, there is an eslint rule to prevent this: https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-cycle.md

This issue will be fixed on version 2, you can follow the progress on #334