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

Decouple react native release #5626

Merged
merged 5 commits into from
Feb 16, 2019
Merged

Conversation

shilman
Copy link
Member

@shilman shilman commented Feb 16, 2019

What I did

This PR excludes @storybook/react-native and @storybook/react-native-server from the 5.0 release process. RN is not ready for a 5.0 release despite heroic efforts from the team.

After it's merged, I will release 5.0.0-rc.0 and announce the RC to get testing feedback so we can release 5.0 and get out of this painful limbo state.

So what happens to RN?

RN development

This PR targets the release/5.0 branch, so in terms of RN development on next, nothing will change. There is a branch https://github.com/storybooks/storybook/tree/react-native/use-core-for-server where a bunch of us are trying to migrate RN server to use Storybook core. Contributions are welcome there: please comment here if you'd like to help maintain Storybook for RN or jump onto our Discord https://discord.gg/qhAxMgN

RN release

I will release Storybook for RN as soon as it's ready and tested, provided that it does not add instability to the 5.0 release process. Most likely I'll release it as part of the Storybook 5.1 release, possibly expediting that to get RN up to date ASAP.

How to test

yarn lerna list

@shilman shilman added the maintenance User-facing maintenance tasks label Feb 16, 2019
@shilman shilman self-assigned this Feb 16, 2019
Copy link
Member

@tmeasday tmeasday left a comment

Choose a reason for hiding this comment

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

WFM let’s get it done

@codecov
Copy link

codecov bot commented Feb 16, 2019

Codecov Report

Merging #5626 into release/5.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/5.0    #5626   +/-   ##
============================================
  Coverage        32.85%   32.85%           
============================================
  Files              638      638           
  Lines             9241     9241           
  Branches          1314     1314           
============================================
  Hits              3036     3036           
  Misses            5565     5565           
  Partials           640      640

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 9c28b66...4007bb7. Read the comment docs.

@shilman shilman removed their assignment Feb 16, 2019
@shilman shilman merged commit 7e619c4 into release/5.0 Feb 16, 2019
@shilman shilman deleted the shilman/decouple-react-native branch February 16, 2019 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants