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
Merge packages data-default-class and data-default-instances-base #7
Comments
+1 I'll add Given that |
I agree with @sol. I have always avoided the As concrete examples, I'd love to use the union of |
@lfairy I think it is preferable to only have BTW, similar to @ndmitchell I would love to use |
What would be the best way to merge them? Move all the code from |
@mauke That seems reasonable to me. |
I may be missing something, but as I see it the reason to split the original
data-default
into separate packages is to minimize dependencies.As both
data-default-class
anddata-default-instances-base
only introduce thebase
dependency, we could merge the packages.Keeping the separate packages has the downside that the instances in
data-default-instances-base
are orphan (which I generally considered undesirable). Are there any upsides?The text was updated successfully, but these errors were encountered: