-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: split files #56
Conversation
I would really like to try to keep the tests close to the source (locality of behaviour mindset), do you think we could consider ways to achieve it? One idea would be to group the utilities by types and then in each folder have all the source + tests:
That might help to make the new naming convention clearer. |
I agree with the comments from @p9f . |
I couldn't think of.a better name for the |
I think the |
Co-authored-by: Guga Guichard <gustavoguichard@gmail.com>
maybe we remove the
|
For the |
For sure, will make those changes during lunch today. If you can think of any other feedback, let me know. |
@jly36963 to avoid conflicts I'll stop my activity until this PR is merged =) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description