Refactoring #451
jankapunkt
started this conversation in
Ideas
Refactoring
#451
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
The recent pull requestst dealt a lot with merge conflicts. This is often a result from large files being edited on multiple branches.
This is to me an indicator of lacking abstraction and tight coupling of the code within these large files.
I think, if we want to improve this package we should do a refactoring into smaller files, for example one module per function/class and build a more injection-based solution.
@lynchem @guncebektas @ricaragao
What do you think?
Beta Was this translation helpful? Give feedback.
All reactions