Skip to content
This repository has been archived by the owner on Jul 19, 2019. It is now read-only.

Add notes for April, 21 #8

Merged
merged 1 commit into from
Apr 22, 2016
Merged

Add notes for April, 21 #8

merged 1 commit into from
Apr 22, 2016

Conversation

gaearon
Copy link
Member

@gaearon gaearon commented Apr 22, 2016

Please use this pull request for the discussion!

@gaearon gaearon merged commit 7419299 into master Apr 22, 2016
@gaearon gaearon deleted the gaearon-patch-1 branch April 22, 2016 13:38
* In six months, master and 15.x can diverge significantly.
* We need to figure out a better story around syncing React with:
* The Facebook website.
* React Native.

Choose a reason for hiding this comment

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

Is there any need to have a better story around syncing React with the outside community? Or is npm and the semver changes seen as sufficient for that front and only internal consumers needs to be tackled now?

Copy link
Member Author

@gaearon gaearon Apr 22, 2016

Choose a reason for hiding this comment

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

Can you explain in a little more detail what you mean by syncing externally?

At Facebook, we have a monorepo, and it is the source of truth for the website. So we literally copy the new files into the Facebook repo from master once a couple of weeks, catch regressions or find bugs in our own components, fix those, and then land the change. It can get reverted if something turns out to break in the internal testing, so this process is manual so we have full control over it and can cherry-pick commits if necessary. Generally, this syncing helps ensure that by the time we cut an open source release, the code is already in production so we don’t have big regressions. (Obviously this didn’t entirely pan out in 15.)

I’m not sure what would be an external equivalent of that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants