Skip to content
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

Support including mixins with @content without passing content. #366

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

alvra
Copy link

@alvra alvra commented May 9, 2017

I have no idea if this is the correct approach but it seems to fix issue #365.

@alvra
Copy link
Author

alvra commented Aug 14, 2017

The test I just added shows the change outputs the mixin twice. Any idea why this is happening?
It does prevent the error and outputs the correct css.

@coveralls
Copy link

coveralls commented Aug 14, 2017

Coverage Status

Coverage decreased (-0.06%) to 66.912% when pulling e70b36f on alvra:patch-1 into 7f4725b on Kronuz:master.

@alvra
Copy link
Author

alvra commented Aug 14, 2017

The latest commit fixes the problem of duplicate output. Travis shows the tests pass, except for two unrelated problems.

Is there anything else I can do to help this along?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants