Skip to content

[webpack-encore-bundle] - update comments in default recipe for new features #562

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
3 commits merged into from
Apr 10, 2019
Merged

[webpack-encore-bundle] - update comments in default recipe for new features #562

3 commits merged into from
Apr 10, 2019

Conversation

PhilETaylor
Copy link
Contributor

Q A
License MIT

This PR makes two changes

  1. update documentation for previous feature allowing output_path to be false

  2. update documentation for new feature to set the value fo crossorigin attribute if PR propose fix for CORS issues with missing crossorigin tag webpack-encore-bundle#56 is accepted

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

@PhilETaylor
Copy link
Contributor Author

ONLY MERGE IF if PR symfony/webpack-encore-bundle#56 is accepted

@PhilETaylor PhilETaylor changed the title webpack - update comments in default recipe for new features [webpack-encore-bundle] - update comments in default recipe for new features Apr 1, 2019
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

@Nyholm Nyholm changed the title [webpack-encore-bundle] - update comments in default recipe for new features WIP: [webpack-encore-bundle] - update comments in default recipe for new features Apr 6, 2019
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

@PhilETaylor
Copy link
Contributor Author

This can be merged as the upstream change has been merged in symfony/webpack-encore-bundle@2384cfd

@weaverryan weaverryan changed the title WIP: [webpack-encore-bundle] - update comments in default recipe for new features [webpack-encore-bundle] - update comments in default recipe for new features Apr 10, 2019
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request does not pass validation.

@fabpot
Copy link
Member

fabpot commented Apr 10, 2019

@PhilETaylor Can you rebase this PR? It will then be merged automatically. Thank you.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

@ghost ghost merged commit 66b2161 into symfony:master Apr 10, 2019
ghost pushed a commit that referenced this pull request Apr 10, 2019
@PhilETaylor
Copy link
Contributor Author

Rebased and merged... 🎉

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants