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

[0.53] Commits to cherry-pick into stable release #17520

Closed
grabbou opened this issue Jan 10, 2018 · 16 comments
Closed

[0.53] Commits to cherry-pick into stable release #17520

grabbou opened this issue Jan 10, 2018 · 16 comments
Labels
Resolution: Locked This issue was locked by the bot.

Comments

@grabbou
Copy link
Contributor

grabbou commented Jan 10, 2018

The thread containing issues that are blocking the 0.53 release and need to be addressed.

Please keep this list to critical bug fixes that are new in 0.53 and ideally have been merged into master -- post commit hashes rather than issue/PR numbers unless the issue/PR in question targets a problem specific to 0.53. The intent of these cherry-pick threads is not to solicit features but to fix things that make 0.53 substantially more launch-blocked than 0.52.

@ide
Copy link
Contributor

ide commented Jan 16, 2018

Cherry-picked 8b64907 (fix HMR client message) for CRNA
29f8354 (fix CORS issue with xip.io)

@Andreyco
Copy link
Contributor

Andreyco commented Jan 29, 2018

Unify boost vs boost-for-react-native dependency, commit here d40db3a
Greatly improves build times.

@vonovak
Copy link
Contributor

vonovak commented Jan 30, 2018

this is a bugfix for a RefreshControl issue introduced in 0.51 (or 0.52?), would be nice to have it fixed already in 0.53: ed5872e

@ide
Copy link
Contributor

ide commented Jan 31, 2018

Cherry-picked in the RN Boost and RefreshControl commits. The Yoga commit does not merge cleanly and affects code that landed after 0.53 was cut.

@grabbou
Copy link
Contributor Author

grabbou commented Jan 31, 2018 via email

@ide
Copy link
Contributor

ide commented Feb 2, 2018

@grabbou I think that’s everything for now in 0.53-stable.

@brunolemos
Copy link
Contributor

Is there a thread for non critical commits?


Context: The PR #16933 got merged to master (2-line change), but the import was messed up and deleted the whole file (0d4ff1b) and then it was recreated in another commit (b8c86b8) which does include the PR mentioned but can easily go unnoticed, so that's why I'm mentioning it here.

@grabbou
Copy link
Contributor Author

grabbou commented Feb 5, 2018

No, we don't @brunolemos. Best to just let me know around the end / beginning of a month when I am about to release a new version.

@grabbou
Copy link
Contributor Author

grabbou commented Feb 5, 2018

Meanwhile, let me close this issue as I am promoting 0.53 to stable.

@grabbou grabbou closed this as completed Feb 5, 2018
@jozan
Copy link
Contributor

jozan commented Feb 15, 2018

I found one bug and fixed it in local-cli. It prevents us from upgrading since it breaks running on actual iOS device. I was directed to try pinging this thread in hopes to get patch release.

PR in question: #17983

@jozan
Copy link
Contributor

jozan commented Feb 15, 2018

Commit: f8fee0a

Sorry for ugly description, I don’t know how all this works...

@MrLoh
Copy link

MrLoh commented Feb 16, 2018

It’s very unfortunate that 0.53 was release while not being stable. Without being able to run on device this is not very usable.

@jozan
Copy link
Contributor

jozan commented Feb 16, 2018 via email

@MrLoh
Copy link

MrLoh commented Feb 17, 2018

Yeah I really appreciate the effort of all the people on making React native work. And 0.53 is a great release. And of course bugs always happen. Developing react native for around 6 months now, I’m just sometimes really frustrated how unstable everything still is (of course it’s beta) but It’s such a popular option for production apps. Whenever I do some web development now I’m amazed by how incredibly stable things are in comparison. Obviously the task for React native is monumental and it will probably take years to reach a similar level of maturity as web development. So everyone who is working towards that has my huge appreciation, it’s sometimes just hard to cope with the frustration of spending many hours again on just getting things to work. I have no idea how to improve this either, I guess we just all have to help where we can and bring a lot more patience and appreciation for the complexity of the problem that react native solves.

Sent with GitHawk

@grabbou
Copy link
Contributor Author

grabbou commented Feb 20, 2018

Thank you @MrLoh and @jozan for taking your time to share your thoughts on the latest stable release. I appreciate your feedback. I have cherry-picked this small fix to unblock you from upgrading. Should be out on npm within next couple of hours.

I agree that React Native has its unstable moments from time to time, but as others successfully pointed out, it is still a fairly new technology. The biggest problem with releases being sometimes prematurely promoted to stable is that the amount of people running on RC channel is limited. That makes it hard to spot problems before hitting stable channel.

We are doing our best to make that as pleasant experience as possible by controlling post-release state of issues and coordinating with Facebook the release status.

@MrLoh
Copy link

MrLoh commented Feb 20, 2018

@grabbou thank you so much for making this release!! We are very grateful for all the hard work that you and so many others put into React Native. RN is a truly amazing technology solving a really hard problem and having a big impact on App development.

@facebook facebook locked as resolved and limited conversation to collaborators Feb 5, 2019
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Feb 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Resolution: Locked This issue was locked by the bot.
Projects
None yet
Development

No branches or pull requests

8 participants