-
Notifications
You must be signed in to change notification settings - Fork 9
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.75] Implement more missing methods on WithRuntimeDecorator #579
Comments
This will most likely require a similar pick on the hermes side, so it's not a trivial one. |
As the two commits were less than a day apart, and is only an implementation of the missing api, theoretically speaking there shouldn't actually be a problem. But we can always wait till 0.76 beta is released and the commit is tested and found stable, no particular hurry on this one. |
Which two commits are you referring to? |
I am referring to these two commits only. The first of the commits is already merged in this [0.75] branch. |
Correct, and we had to bump hermes for that specific commit. We'll have to do the same for this other commit if we decide to include it |
Noted above. Adding the corresponding hermes commit-ish for the should bump hermes (?) check if you do decide to cherry-pick. |
Examining the compare of two consecutive hermes commit tags hermes engine RN-0.75.1 vs 0.76.0, it may be possible the previous bump to Hermes might not have actually happened either. Is there any way to query which hermes engine is actually running? |
✅ Picked |
Target Branch(es)
0.75
Link to commit or PR to be picked
facebook/react-native@993f9fd
Description
This Implement more missing methods on WithRuntimeDecorator is a followup pick to the already selected cherrypick in 0.75 Add missing WithRuntimeDecorator methods and should probably be added for completeness
According to the author of the commit these missing methods unfortunately threw no compile time errors due to inheritance.
The text was updated successfully, but these errors were encountered: