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

fix: storybook & webpack 5 successfully build #12964

Closed
wants to merge 19 commits into from

Conversation

ScriptedAlchemy
Copy link

@ScriptedAlchemy ScriptedAlchemy commented Oct 31, 2020

Issue: Build didn't work

What I did

  • updated fork ts plugin
  • updated webpack to v5 stable
  • implemented mjs bandaid loader
  • updated loader options API
  • kind of fixed cache issues, but needs improvement
  • updated webpack virtual modules
  • fixed incorrect devtools

How to test

i ran one example and it seems to build. needs more testing from those who know storybook

If your answer is yes to any of these, please make sure to include it in your PR.

Snapshot Summary
 › 5 snapshots failed from 2 test suites. Inspect your code changes or re-run jest with `-u` to update them.

Test Suites: 5 failed, 121 passed, 126 total
Tests:       9 failed, 1452 passed, 1461 total
Snapshots:   5 failed, 416 passed, 421 total
Time:        75.798 s
Ran all test suites in 7 projects.

there are many references to webpack 4 in here. If I do a global search, everything needs to updated to webpack 5. You can use syncpack to sync it every

@ScriptedAlchemy ScriptedAlchemy changed the title fix: Webpack 5 successfully builds fix: storybook & webpack 5 successfully build Oct 31, 2020
# Conflicts:
#	addons/a11y/package.json
#	addons/actions/package.json
#	addons/backgrounds/package.json
#	addons/controls/package.json
#	addons/cssresources/package.json
#	addons/design-assets/package.json
#	addons/docs/package.json
#	addons/essentials/package.json
#	addons/events/package.json
#	addons/google-analytics/package.json
#	addons/graphql/package.json
#	addons/jest/package.json
#	addons/knobs/package.json
#	addons/links/package.json
#	addons/queryparams/package.json
#	addons/storyshots/storyshots-core/package.json
#	addons/storyshots/storyshots-puppeteer/package.json
#	addons/storysource/package.json
#	addons/toolbars/package.json
#	addons/viewport/package.json
#	app/angular/package.json
#	app/aurelia/package.json
#	app/ember/package.json
#	app/ember/src/server/framework-preset-babel-ember.ts
#	app/html/package.json
#	app/marionette/package.json
#	app/marko/package.json
#	app/mithril/package.json
#	app/preact/package.json
#	app/rax/package.json
#	app/react/package.json
#	app/riot/package.json
#	app/server/package.json
#	app/svelte/package.json
#	app/vue/package.json
#	app/web-components/package.json
#	dev-kits/addon-decorator/package.json
#	dev-kits/addon-parameter/package.json
#	dev-kits/addon-roundtrip/package.json
#	docs/package.json
#	examples/angular-cli/package.json
#	examples/aurelia-kitchen-sink/package.json
#	examples/cra-ts-essentials/package.json
#	examples/cra-ts-kitchen-sink/package.json
#	examples/dev-kits/package.json
#	examples/marko-cli/package.json
#	examples/mithril-kitchen-sink/package.json
#	examples/official-storybook/package.json
#	examples/preact-kitchen-sink/package.json
#	examples/rax-kitchen-sink/package.json
#	examples/react-ts/package.json
#	examples/riot-kitchen-sink/package.json
#	examples/server-kitchen-sink/package.json
#	examples/vue-cli/package.json
#	examples/vue-kitchen-sink/package.json
#	examples/web-components-kitchen-sink/package.json
#	lib/addons/package.json
#	lib/api/package.json
#	lib/api/src/version.ts
#	lib/channel-postmessage/package.json
#	lib/channel-websocket/package.json
#	lib/channels/package.json
#	lib/cli/package.json
#	lib/cli/test/fixtures/react_static_next/package.json
#	lib/cli/test/fixtures/sfc_vue/package.json
#	lib/codemod/package.json
#	lib/components/package.json
#	lib/core/package.json
#	lib/core/src/server/preview/iframe-webpack.config.js
#	lib/source-loader/package.json
#	lib/theming/package.json
#	lib/ui/package.json
#	package.json
#	yarn.lock
@ndelangen
Copy link
Member

@ScriptedAlchemy I tried to merge next into this branch to help you out, but stumbled on quite a few merge conflicts. I tried to resolve them as best as I can. Maybe this help, maybe not:

https://github.com/storybookjs/storybook/tree/pr/ScriptedAlchemy/12964-2

@ScriptedAlchemy ScriptedAlchemy changed the base branch from tech/webpack5 to next November 3, 2020 05:01
@ydmitry
Copy link

ydmitry commented Nov 19, 2020

Any updates on this?

@ndelangen ndelangen mentioned this pull request Dec 7, 2020
3 tasks
@stale
Copy link

stale bot commented Dec 25, 2020

Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 30 days. Thanks!

@stale stale bot added the inactive label Dec 25, 2020
@phated
Copy link
Contributor

phated commented Feb 22, 2021

Webpack 5 support is available in 6.2 Prerelease version now! See the testing instructions at https://gist.github.com/shilman/8856ea1786dcd247139b47b270912324 and if you have any issues, please open a new issue about it so we can take a look!

@phated phated closed this Feb 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants