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.75] Implement more missing methods on WithRuntimeDecorator #579

Open
joe-sam opened this issue Oct 15, 2024 · 7 comments
Open

[0.75] Implement more missing methods on WithRuntimeDecorator #579

joe-sam opened this issue Oct 15, 2024 · 7 comments

Comments

@joe-sam
Copy link

joe-sam commented Oct 15, 2024

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.

@cortinico
Copy link
Collaborator

This will most likely require a similar pick on the hermes side, so it's not a trivial one.

@joe-sam
Copy link
Author

joe-sam commented Oct 15, 2024

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.

@cortinico
Copy link
Collaborator

As the two commits were less than a day apart

Which two commits are you referring to?

@joe-sam
Copy link
Author

joe-sam commented Oct 15, 2024

I am referring to these two commits only. The first of the commits is already merged in this [0.75] branch.

@cortinico
Copy link
Collaborator

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

@joe-sam
Copy link
Author

joe-sam commented Oct 16, 2024

Noted above.

Adding the corresponding hermes commit-ish for the should bump hermes (?) check if you do decide to cherry-pick.

facebook/hermes@1af8c8

@joe-sam
Copy link
Author

joe-sam commented Oct 17, 2024

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Inbox
Development

No branches or pull requests

2 participants