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
Remove beforeLint #35
Milestone
Comments
I am ok with dropping it. @timkelty ? |
OK with me. |
Gonna pick this up |
simonsmith
added a commit
that referenced
this issue
Sep 22, 2016
This was an underused feature and adds complexity to the preprocessor. This could be solved more elegantly by allowing the user to pass their own `transform` function to postcss-import. Fixes #35
Merged
simonsmith
added a commit
that referenced
this issue
Sep 22, 2016
This was an underused feature and adds complexity to the preprocessor. This could be solved more elegantly by allowing the user to pass their own `transform` function to postcss-import. Fixes #35 Improve variable naming This makes it a little clearer what `merged` is Remove superfluous comments These are too basic to need comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I feel like this feature is probably unused and adds a lot of complexity to the preprocessor. I think it would also fix #27.
The only use case I had before was to unwrap nested rules for the
bem-linter
but it handles that out of the box now.Thoughts?
The text was updated successfully, but these errors were encountered: