Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Reviewed the Assetic cookbook articles#5094
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
javiereguiluz commentedMar 19, 2015
| Q | A |
|---|---|
| Doc fix? | yes |
| New docs? | no |
| Applies to | all |
| Fixed tickets | - |
| Now you can execute the ``uglifyjs`` command that lives in the ``node_modules`` | ||
| directory: | ||
| .. code-block:: bash | ||
| $ ./app/Resources/node_modules/.bin/uglifyjs --help |
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.
for windows compatbility, the path should be in double quotes
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.
You mean like this?
$ "./app/Resources/node_modules/.bin/uglifyjs" --helpThere 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.
yes
2015-03-19 11:41 GMT+01:00 Javier Eguiluznotifications@github.com:
In cookbook/assetic/uglifyjs.rst
#5094 (comment):
- $ uglifyjs --help
+Now you can execute theuglifyjscommand that lives in thenode_modules
+directory:+.. code-block:: bash
$ ./app/Resources/node_modules/.bin/uglifyjs --help
You mean like this?
$ "./app/Resources/node_modules/.bin/uglifyjs" --help
—
Reply to this email directly or view it on GitHub
https://github.com/symfony/symfony-docs/pull/5094/files#r26743251.
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.
Done. Thanks.
👍 |
| @@ -204,7 +204,7 @@ Combining Assets | |||
| ~~~~~~~~~~~~~~~~ | |||
| One feature of Assetic is that it will combine many files into one. This helps | |||
| to reduce the number of HTTP requests, which is great forfront end performance. | |||
| to reduce the number of HTTP requests, which is great forfrontend performance. | |||
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.
Not sure about this change. While I agree that "frontend" looks most beautiful, it seems that "front-end" actually would be more correct (see also#4671 (comment)).
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.
both Bower and Bootstrap (on their homepages) usefront-end. That looks a bit odd to me, but I guess we should follow their lead.
* 2.3: (44 commits) Removed duplicate "long"s terminate file with newline move title back and move description to separate file as suggested in PR#4892 move title out of included file as suggested in PR#4892 Add possible values for widget_type [#4842] Making 2 sentences changes as suggested by WouterJ in pull request#4842 Add meaning of yellow icon for number of queries Fixing bad link name fix typo in event flow diagrams Many fixes thanks to stof, WouterJ, xabbuh and dupuchba added Jakub as a merger for the DomCrawler component [#5094] Removing mkdir - it's not needed (thanks xabbuh) some tweaks to#4601 Moving index down to correct section [#4989] Language tweaks and making the example simpler Remove useless setLocale() call and add code block with locale setter Finaly touches on translation locale setting note Review note about setting the translator locale in a controller. Update translation.rst ...Conflicts:cookbook/security/entity_provider.rst
* 2.6: (45 commits) Removed duplicate "long"s terminate file with newline move title back and move description to separate file as suggested in PR#4892 move title out of included file as suggested in PR#4892 Add possible values for widget_type [#4842] Making 2 sentences changes as suggested by WouterJ in pull request#4842 Add meaning of yellow icon for number of queries Fixing bad link name fix typo in event flow diagrams Many fixes thanks to stof, WouterJ, xabbuh and dupuchba added Jakub as a merger for the DomCrawler component [#5094] Removing mkdir - it's not needed (thanks xabbuh) some tweaks to#4601 Moving index down to correct section [#4989] Language tweaks and making the example simpler Remove useless setLocale() call and add code block with locale setter Finaly touches on translation locale setting note Review note about setting the translator locale in a controller. Update translation.rst ...
* 2.7: (47 commits) Removed duplicate "long"s terminate file with newline move title back and move description to separate file as suggested in PR#4892 move title out of included file as suggested in PR#4892 Add possible values for widget_type [#4842] Making 2 sentences changes as suggested by WouterJ in pull request#4842 Add meaning of yellow icon for number of queries Fixing bad link name fix typo in event flow diagrams Many fixes thanks to stof, WouterJ, xabbuh and dupuchba added Jakub as a merger for the DomCrawler component Changes thanks to WouterJ and xabbuh [#5094] Removing mkdir - it's not needed (thanks xabbuh) some tweaks to#4601 Moving index down to correct section [#4989] Language tweaks and making the example simpler Remove useless setLocale() call and add code block with locale setter Finaly touches on translation locale setting note Review note about setting the translator locale in a controller. ...