Upcoming v0.64.3 cherrypicks discussion

[kelset]

Conversations on this thread are limited to 0.64 releases major issues and backport (cherry-pick) requests from commits that are already on master.

An example of a good such request is a bug fix for a serious issue that has been merged into master but did not make the 0.64.2 cut, with a link to the specific commit hash on master with the commit to cherry-pick, like this example link: facebook/react-native@bd2b7d6

In other words, if you cannot point to a particular commit on master, then your request likely belongs as a new issue in http://github.com/facebook/react-native/issues.

[omnikron]

Would it be possible to include facebook/react-native@bdfe2a5 , which fixes an issue with broken iOS builds (facebook/react-native#31193)? The commit is in master since 16th April but we are facing the same issue today after upgrading to 0.64.2. We added patch-package to apply the commit on top of 0.64.0 every time we build our app and can confirm that it works.

[danilobuerger]

@omnikron @kelset wasn't this already requested to be cherry picked in 0.64.2?

[omnikron]

@omnikron @kelset wasn't this already requested to be cherry picked in 0.64.2?

Yes, but it was not included due to merge conflicts. I didn't find the reference before I wrote my previous comment but it's here: #224 (comment)

Not sure whether the list of unmerged cherry-picks carries over from release to release - apologies if so.

[kelset]

We will also cherry pick this: facebook/react-native#31682

re: this commit facebook/react-native@bdfe2a5 as pointed out couldn't get cherry picked cleanly, so it's not bound to happen as is. What would need to happen is to have a dedicated PR that targets 0.64 and replicates the changes needed in a way that goes in cleanly.

[danilobuerger]

I have it as a commit in my fork that cleanly cherry-picks onto 0.64.2: feastr/react-native@d9f7770 if anybody wants to PR it to the 0.64 branch, feel free.

[omnikron]

I have it as a commit in my fork that cleanly cherry-picks onto 0.64.2: feastr/react-native@d9f7770 if anybody wants to PR it to the 0.64 branch, feel free.

I've done just that: facebook/react-native#31688

@kelset is this sufficient for cherry-picking? Happy to help in any other way.

[kelset]

We also want facebook/react-native@4d40b53 IIRC

[andreialecu]

I hope this is not dumb to ask for here, but updating to 0.65 will be a hard task for a lot of people, and this small fix would be incredible to have as part of 0.64.

Font weights have been broken on Android since forever, and this finally fixes them. It's been merged recently:

facebook/react-native#29117 (facebook/react-native@3827ca6)

[birkir]

Maybe we can include accessible colors for DynamicColorIOS in 64.3? facebook/react-native@4b9d9dd

[dskrzypczynski]

I think this commit is very important: facebook/react-native@70da640

[mikehardy]

@Attimo91 JCenter is actually not that important right now:

https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/

UPDATE 4/27/2021: We listened to the community and will keep JCenter as a read-only repository indefinitely. Our customers and the community can continue to rely on JCenter as a reliable mirror for Java packages.

...slowly exhale on that one. It should be removed, but zero need for cherry-picking anywhere I think.

[jasperjorna]

Is there a predicted release date for v0.64.3?

[mikehardy]

0.64.3 may never happen @jasperjorna - 0.65.0 is in RC2 phase and may release shortly, getting it out takes priority I believe

[geraintwhite]

facebook/react-native@2f477b1 allows jest 27 to work with react-native. It would be good to include if we do get a 0.64.3.

[mikehardy]

@grit96 this is even more important to note over on the 0.65.0 RC2 phase issue as it is not on the release-0.65 branch and that's where current activity is, can you note that over there? #235

[zgordon01]

Any updates on if we are skipping 0.64.3 in favor of 0.65.0?

[kelset]

not much to update from my side sadly, been too busy with work stuff. The plan was for me to onboard another releaser and have them do 0.64.3 as their first "guided" release, but that fell through because this person was not able to get time at work to dedicate to this.

At this point we want to get to 0.65.0; then we'll see how hard/complicated it is to update (upgrade-helper seems to suggest it's not going to be hard) and then decide if we need a patch for 0.64.

[taylorkline]

facebook/react-native@58444c7 applies cleanly and allows for optionally providing custom NSURLSession configurations. If a patch release of 0.64 is made, this would be a nice inclusion for certain use cases.

[nextlevelbeard]

I believe the test automation community would highly appreciate these:
facebook/react-native@efb359f (facebook/react-native#31865)
facebook/react-native@91cac20 (facebook/react-native#31942)

Small but important fixes to the testID prop on Android builds.
It is currently (0.64) not working with certain important components like TextInput and ScrollView.

[mikehardy]

@nextlevelbeard at this point you are really going to want to post these on #244 since they do not appear to be on the release 65 branch or in a release yet. I doubt 64 will see more releases

[nextlevelbeard]

Thanks @mikehardy, it is done

[kelset]

Hey folks, we had a meeting last night and we decided that since 0.66 is already in the work (#246), and that moving to 0.65 is fairly straightforward (upgrade-helper) we won't be doing any more patches (unless something massive happens) for 0.64.