-
Notifications
You must be signed in to change notification settings - Fork 18
Add multi-line imports #42
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
Any reason why this PR has not yet been approved? This is indeed very helpful if you import a lot of values from another file :) |
would love to see this approved ASAP. Multi-line imports is a must for serious theming :-/ |
@ai Is there anything we could do to get the repo owner's attention? It's the css modules core plugins. They should definitely be maintained by someone... :-/ |
@geelen ^ |
@andywer perhaps make an own fork - the remedy of last resort? |
@witwit As a last resort, sure. But I would rather not if there is a way around... Maybe set some random deadline just for ourselves? Like two weeks and if there is still no solution at the time then we fork. |
@andywer, sounds like a plan :-) |
Any progress on this? |
Sorry! This has been not-really-maintained for a while now. But I'm back. |
Great! Thank you |
This adds support for multi-line imports wrapped in parentheses: #41
I added a test for:
The following are also allowed:
Let me know if you'd prefer to change anything or would like to add or re-organize the tests somehow. I can also add to the
README.md
as well.