-
Notifications
You must be signed in to change notification settings - Fork 95
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
Upgrade to use patternlab core 3.x #13
Comments
Should also include pattern-lab/patternlab-node#745 |
Wouldn't it be better to separate the upgrade to Webpack 4 to a separate task? Sure these are both breaking changes but still, in case one or the other gets stuck, you're still open to create a release independently. Also, I think I could more easily contribute to the Webpack part if needed, but not to the patternlab core one. |
@renestalder Thank you, I've split it up, #16 is dedicated to that work. This is a great suggestion thank you. The more contributors the better. |
First off, I might agree with @renestalder's suggestion. That said ;) ... If you are interested, I took a stab at a fork with PL v3 and Webpack v4, but it's rough around the edges. I can say it runs and builds a project I've been working on separately using the release version of patternlab-edition-node-webpack. I'd be interested to get your input, or just have a look at it if you're interested. A few notes
|
Thank you @Josh68 I will take a look as soon as I can. |
FWIW, I think my fork is not a good reference. I've been re-reading the upgrade docs for PL v3, and I'm now trying to wrap my head around what Webpack will handle, and how, and it's substantially different with v3 than with v2. My own goal is to have the ability to use loaders, primarily for SASS/Postcss, and plugins. but with PL core handling file copying and potentially dev serving and watching, I've kind of lost a grasp on how to configure Webpack to do a more minimal set of tasks, at the right time and in the right order. Would it make sense to define goals for a v3 conversion? |
Let's connect on gitter @Josh68 |
Now with upgrading of Webpack to version 4, this prepares this edition to work better with the core 3.x release. |
Upgrade the edition to support https://github.com/pattern-lab/patternlab-node/releases latest release.
Which includes pattern-lab/patternlab-node#745
The text was updated successfully, but these errors were encountered: