Skip to content
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

Framework: Use CustomTemplatedPathPlugin updated for Webpack 4 #5687

Merged
merged 1 commit into from
Mar 21, 2018

Conversation

aduth
Copy link
Member

@aduth aduth commented Mar 18, 2018

Merges to try/webpack4 (#5267)
Related: WordPress/packages#93

This pull request seeks to update the custom CustomTemplatedPathPlugin Webpack plugin to point to the newly-extracted-and-updated-for-Webpack-4 @wordpress/custom-templated-path-webpack-plugin.

Testing instructions:

Since @wordpress/custom-templated-path-webpack-plugin is not yet published, and it's not possible to define an npm package as a subdirectory of a monorepo, you must use npm's symlink feature to test from the new package.

In packages:

  1. Clone WordPress/packages if you don't already have a local copy
  2. cd packages
  3. git checkout add/custom-templated-path-plugin
  4. npm ln

In Gutenberg:

  1. git checkout update/webpack-custom-templated-path
  2. npm install
  3. npm ln @wordpress/custom-templated-path-webpack-plugin
  4. npm run build

There should be no errors (but perhaps some warnings related to the Webpack 4 upgrade).

@aduth aduth added the Framework Issues related to broader framework topics, especially as it relates to javascript label Mar 18, 2018
@aduth aduth requested a review from youknowriad March 20, 2018 12:09
@aduth aduth force-pushed the update/webpack-custom-templated-path branch from 65a8584 to 42be2fc Compare March 21, 2018 23:06
@aduth
Copy link
Member Author

aduth commented Mar 21, 2018

With @wordpress/custom-templated-path-webpack-plugin having been published, I've rebased the branch and will merge [into try/webpack4] (#5267)

@aduth aduth merged this pull request into try/webpack4 Mar 21, 2018
@aduth aduth deleted the update/webpack-custom-templated-path branch March 21, 2018 23:07
aduth pushed a commit that referenced this pull request Mar 30, 2018
* Webpack 4: Try migrating to WebPack 4

* Framework: Use CustomTemplatedPathPlugin updated for Webpack 4 (#5687)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues related to broader framework topics, especially as it relates to javascript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant