-
Notifications
You must be signed in to change notification settings - Fork 62
Fixed preserve option inside at-rules #43
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
Conversation
|
Thanks for the PR @muftiev 😀 Please add some tests that previously fail and now work |
|
Updated tests with new cases 😄 |
|
@muftiev Let's split those off into their own |
|
Ok, done) |
|
@MadLittleMods Will you accept it? 😄 |
|
@muftiev Most likely, will try to get this in by Monday. Feel free to ping |
|
@MadLittleMods Ping to remind you to deal with this PR |
|
@MadLittleMods Really wish to use your module in my project, but I need this fix 😄 |
|
Sorry for the wait, I brushed across this tab in my browser everyday (on my mind). @muftiev Thanks again for the contribution, I'll get this onto a release pronto. |
|
Thank you! |
|
@muftiev v0.5.2 now pushed to npm, https://www.npmjs.com/package/postcss-css-variables |
|
Great! 😄 |
Fix #42