You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm filing this issue because the MainModule just reached 1k+ lines, I had to add a linter exception for it.
Since MainModule encapsulates all DI bindings for the application, it has grown extremely large and somewhat difficult to navigate.
We should modularize the bindings by domain and install them from within the MainModule. Things should also keep package boundaries neat since nobody other than the domain-specific module should have to import the xxx.impl packages.
I'm filing this issue because the
MainModule
just reached 1k+ lines, I had to add a linter exception for it.Since MainModule encapsulates all DI bindings for the application, it has grown extremely large and somewhat difficult to navigate.
We should modularize the bindings by domain and
install
them from within the MainModule. Things should also keep package boundaries neat since nobody other than the domain-specific module should have to import thexxx.impl
packages.Examples:
AlertModule, ReportingModule, RollupModule
The text was updated successfully, but these errors were encountered: