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

chore: remove storybook, chromatic #716

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

WilliamKelley
Copy link
Contributor

@WilliamKelley WilliamKelley commented Nov 16, 2023

This must come before removing Spark so that the Chromatic is already removed. It's a separate concern from removing the spark stuff

  • delete chromatic workflow
  • delete root storybook config: /.storybook/'
  • uninstall root storybook config deps: npm uninstall @storybook/addon-essentials
  • delete spark lib storybook config: /libs/spark/.storybook/
  • uninstall spark lib storybook deps:
    • explicit: npm uninstall @storybook/addon-links storybook-addon-pseudo-states @storybook/react
    • implicit: npm uninstall @storybook/manager-webpack5 @storybook/builder-webpack5 @svgr/webpack @types/webpack webpack
    • dangling: nom uninstall @storybook/addon-storyshots
  • delete nx storybook stuff
    • workspace spark project targets: storybook, build-storybook
  • uninstall nx storybook stuff: nom uninstall @nrwl/storybook
  • delete /libs/spark/public/ (used for storybook asset storage)
  • delete /libs/spark/stories/ (used for story utils)

@WilliamKelley WilliamKelley force-pushed the wk/20231115-chore-remove-storybook-chromatic branch from 01961c6 to 49bbc35 Compare November 16, 2023 00:30
@WilliamKelley WilliamKelley changed the base branch from wk/20231115-feat-remove-scripts to wk/20231115-feat-remove-spark-icons November 16, 2023 00:30
@WilliamKelley WilliamKelley force-pushed the wk/20231115-chore-remove-storybook-chromatic branch from 49bbc35 to f55d834 Compare November 16, 2023 00:39
Base automatically changed from wk/20231115-feat-remove-spark-icons to main November 20, 2023 21:27
- delete chromatic workflow
- delete root storybook config: `/.storybook/`'
- uninstall root storybook config deps: `npm uninstall @storybook/addon-essentials`
- delete spark lib storybook config: `/libs/spark/.storybook/`
- uninstall spark lib storybook deps:
  - explicit: `npm uninstall @storybook/addon-links storybook-addon-pseudo-states @storybook/react`
  - implicit: `npm uninstall @storybook/manager-webpack5 @storybook/builder-webpack5 @svgr/webpack @types/webpack webpack`
  - dangling: `nom uninstall @storybook/addon-storyshots`
- delete nx storybook stuff
  - workspace spark project targets: storybook, build-storybook
- uninstall nx storybook stuff: `nom uninstall @nrwl/storybook`
- delete `/libs/spark/public/` (used for storybook asset storage)
- delete `/libs/spark/stories/` (used for story utils)
@WilliamKelley WilliamKelley force-pushed the wk/20231115-chore-remove-storybook-chromatic branch from f55d834 to f8c7361 Compare November 20, 2023 21:28
@WilliamKelley WilliamKelley merged commit c6c958d into main Nov 20, 2023
@WilliamKelley WilliamKelley deleted the wk/20231115-chore-remove-storybook-chromatic branch November 20, 2023 21:29
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.

2 participants