-
Notifications
You must be signed in to change notification settings - Fork 126
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
Reorganize Masonry modules #848
Conversation
e60f5a3
to
5afc415
Compare
First commit just moves modules around and adds temporary re-exports in |
This PR was executed in roughly four steps:
In theory there shouldn't be any code changes, though in practice I can't guarantee it. |
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.
If we want to approve this design, I don't see any issues. Daniel suggested splitting a few things up, so it's worth considering that, too.
8e98c41
to
e18d129
Compare
No description provided.