Skip to content

Adding documentation for 0.13.0 Encore feature changes #8272

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

Merged
merged 1 commit into from
Aug 14, 2017
Merged

Adding documentation for 0.13.0 Encore feature changes #8272

merged 1 commit into from
Aug 14, 2017

Conversation

weaverryan
Copy link
Member

@@ -70,6 +70,11 @@ To build the assets, use the ``encore`` executable:
# compile assets, but also minify & optimize them
$ ./node_modules/.bin/encore production

# shorter version of the above 3 commands
$ yarn run encore dev
$ yarn run encore dev -- --watch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this double -- correct?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is. Adding the first one tells yarn to pass the rest of the arguments to Encore instead of directly using them (ref.).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, it's the only downside to the shorter syntax

@weaverryan weaverryan merged commit 58b5596 into symfony:3.3 Aug 14, 2017
weaverryan added a commit that referenced this pull request Aug 14, 2017
…(weaverryan)

This PR was merged into the 3.3 branch.

Discussion
----------

Adding documentation for 0.13.0 Encore feature changes

Various changes, all mentioned here: https://github.com/symfony/webpack-encore/releases/tag/v0.13.0 :)

Commits
-------

58b5596 Adding documentation for 0.13.0 Encore feature changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants