-
Notifications
You must be signed in to change notification settings - Fork 553
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
Added support for Packer #173
Conversation
*/ | ||
class PackerFilter implements FilterInterface | ||
{ | ||
protected $encoding = 'None'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use spaces for the indentation, not tabs
Done! Thanks for the feedback. |
Looks good to me. Could you please add a test for this filter? This is a good example: |
Is still something missing? :) |
Sorry, I wasn't notified when you added this commit. I will review soon. Thanks! |
github never notifies when new commits are pushed in a PR. |
oui |
Oh, this american accent :-D |
Good to know. I didn't know that, sorry. |
Yeah, it's a good idea to post a comment when you push to a PR to notify the folks subscribed. |
@kriswallsmith is there anything missing according to you ? |
Please add the |
And please add it in the Travis setup (you will need to rebase your branch to have it) so that Travis clones the needed repo and can run the test |
Okay, seems that I need some help here... I have done a rebase on my local repository:
But now I can't push it to GitHub...
What should I do now? |
you need to force the push of the branch. A simple push will fail and this is logical as you rewrote the history (git tries to avoid you doing mistakes if you did not want to rewrite it)
|
@maximilian-walter ping. Could you rebase your branch ? Btw, it would be great to add Packer to the vendors downloaded during the travis setup (and adding the path in the travis phpunit config) so that the test does not need to be skipped. |
@maximilian-walter ping |
Sorry, I was very busy the last time. I could rebase my branch and added Packer to the Travis setup, thanks for your help. Can you check it, please. |
@@ -11,5 +11,6 @@ before_script: | |||
- git clone https://github.com/leafo/lessphp.git vendor/lessphp --quiet --depth 1 | |||
- git clone https://github.com/mrclay/minify.git vendor/minify --quiet --depth 1 | |||
- svn checkout http://cssmin.googlecode.com/svn/trunk/ vendor/cssmin --quiet | |||
- wget --quiet -O javascript-packer.zip "http://joliclic.free.fr/php/javascript-packer/telechargement.php?id=2&action=telecharger" && mkdir -p vendor/packer && unzip -qq javascript-packer.zip -d vendor/packer; rm javascript-packer.zip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrong indentation (which will break things as it makes the YAML invalid)
The wrong indentation is fixed! |
Thanks |
Can someone help me to configure this filter in Symfony2 ? |
I added the Filter
PackerFilter
which uses the PHP-Port of Dean Edwards's Packer to compress JavaScript.