Skip to content

Added flow support to getstorybook upgrade#1289

Merged
shilman merged 4 commits intostorybookjs:masterfrom
AugustinLF:add-parser-option-getstorybook
Jun 17, 2017
Merged

Added flow support to getstorybook upgrade#1289
shilman merged 4 commits intostorybookjs:masterfrom
AugustinLF:add-parser-option-getstorybook

Conversation

@AugustinLF
Copy link
Copy Markdown
Contributor

This does work for me on a large flow typed codebase. I'm not sure if there's a cleaner way to pass the options to the generator, if you have any suggestion, don't hesitate, I'll update the PR!

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 15, 2017

Codecov Report

Merging #1289 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1289   +/-   ##
=======================================
  Coverage   13.76%   13.76%           
=======================================
  Files         201      201           
  Lines        4620     4620           
  Branches      508      510    +2     
=======================================
  Hits          636      636           
- Misses       3537     3547   +10     
+ Partials      447      437   -10
Impacted Files Coverage Δ
lib/cli/bin/generate.js 0% <ø> (ø) ⬆️
addons/knobs/src/components/WrapStory.js 12% <0%> (ø) ⬆️
app/react/src/server/build.js 0% <0%> (ø) ⬆️
...codemod/src/transforms/update-organisation-name.js 40.62% <0%> (ø) ⬆️
addons/info/src/components/markdown/code.js 0% <0%> (ø) ⬆️
app/react/src/client/preview/render.js 0% <0%> (ø) ⬆️
app/react/src/client/preview/error_display.js 0% <0%> (ø) ⬆️
lib/components/src/demo/Welcome.js 0% <0%> (ø) ⬆️
addons/knobs/src/components/types/Select.js 7.93% <0%> (ø) ⬆️
app/react-native/src/preview/story_kind.js 0% <0%> (ø) ⬆️
... and 19 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29cc9d8...4d2fd0c. Read the comment docs.

@shilman
Copy link
Copy Markdown
Member

shilman commented Jun 15, 2017

@AugustinLF awesome -- great addition!! can you provide a simple example project to go along with it?

@AugustinLF
Copy link
Copy Markdown
Contributor Author

Here is an example. getstorybook throws an error with the latest published version, while running fine when using my version with getstorybook --parser flow.

Do you want me to rebase my branch on master?

@shilman
Copy link
Copy Markdown
Member

shilman commented Jun 16, 2017

I'll review in the next few hrs and will ask if I need anything 🙇

Copy link
Copy Markdown
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

Awesome!! 🚀 🌔

@shilman shilman changed the title Add --parser option to getstorybook Added flow support to getstorybook upgrade Jun 17, 2017
@shilman shilman merged commit 0942a09 into storybookjs:master Jun 17, 2017
@AugustinLF AugustinLF deleted the add-parser-option-getstorybook branch June 17, 2017 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants