Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Remove postprocessors from unprocessed asset #66

Merged
merged 1 commit into from Aug 16, 2013

Conversation

Projects
None yet
2 participants
Contributor

RavWar commented May 11, 2013

Unprocessed assets are evaluated using all postprocessors but without any engines. It causes unexpected behavior and errors because postprocessor expects data allready parsed by an engine.

I think simply removing postprocessors and leaving just preprocessors and engines for unprocessed assets shouldn't cause any problems.

I found this issue while using https://github.com/ai/autoprefixer-rails. Great gem and i really hope it gets the support it deserves, including compatibility with this also very useful gem.

Owner

ndbroadbent commented Aug 16, 2013

Looks good to me. Will release 0.3.8 with this change

ndbroadbent added a commit that referenced this pull request Aug 16, 2013

Merge pull request #66 from RavWar/postprocessor_fix
Remove postprocessors from unprocessed asset

@ndbroadbent ndbroadbent merged commit 2173aee into ndbroadbent:master Aug 16, 2013

1 check passed

default The Travis CI build passed
Details
Owner

ndbroadbent commented Aug 19, 2013

Hey, I think this change has caused #73... reverting for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment