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) Use standalone proptypes #5988

Merged
merged 6 commits into from
Aug 30, 2017
Merged

(chore) Use standalone proptypes #5988

merged 6 commits into from
Aug 30, 2017

Conversation

MaxBittker
Copy link
Contributor

in preparation for react 16

@MaxBittker MaxBittker requested a review from billyvg August 29, 2017 01:21
@ghost
Copy link

ghost commented Aug 29, 2017

1 Warning
⚠️ You should update CHANGES due to the size of this PR

Generated by 🚫 danger

@MaxBittker
Copy link
Contributor Author

interesting how this broke storybook!

@billyvg
Copy link
Member

billyvg commented Aug 29, 2017

Wonder if this is related? storybookjs/storybook#1723

@MaxBittker
Copy link
Contributor Author

off the cuff instinct is no, gonna investigate further.
this issue seems more accurate but is supposedly fixed in 3.1.4 and we're on 3.2.0
storybookjs/storybook#954

@MaxBittker
Copy link
Contributor Author

storybookjs/storybook#1725

this PR got merged that fixes the issue, gonna try running storybook from master lol

@MaxBittker MaxBittker force-pushed the standalone-proptypes branch from 0e71718 to c7f82be Compare August 30, 2017 18:39
@MaxBittker
Copy link
Contributor Author

MaxBittker commented Aug 30, 2017

upgrading storybook to ^3.2.0 fixed this - want me to pin a specific patch? this is ready to review (and merge?) now i believe @billyvg. thoughts on the percy results?

@MaxBittker MaxBittker changed the title use standalone proptypes (chore) use standalone proptypes Aug 30, 2017
@MaxBittker MaxBittker changed the title (chore) use standalone proptypes (chore) Use standalone proptypes Aug 30, 2017
@MaxBittker MaxBittker merged commit 3935302 into master Aug 30, 2017
@billyvg billyvg deleted the standalone-proptypes branch September 6, 2017 20:35
@github-actions github-actions bot locked and limited conversation to collaborators Dec 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants