Fixes CSS preprocessor - Githubissues

CHH / pipe

Put your assets into the pipe and smoke them.

MIT License

120 stars 9 forks source link

Fixes CSS preprocessor #21

Closed svperfecta closed 12 years ago

svperfecta commented 12 years ago

Hey @chh - When playing with PIPE today I found I couldn't get CSS directives to work at all. (I'm on PHP 5.3.16 if it matters)

The culprit was a mistake in a regular expression that accounts for multi-line statements. I fixed that (small change) and found that the resulting header had a bunch of blank lines in it. I'm stripping them out too. Finally, I added a new unit test to cover the bug.

Cheers! Brian

CHH commented 12 years ago

Apart from a nitty gritty detail, this PR is fine :smile:. Thanks for the great work!

svperfecta commented 12 years ago

Agh, I meant to delete that debug output. I'll do so later.

CHH commented 12 years ago

No problem, I've already done that :smile: