Skip to content

Rename jsi/hermes.h to jsi/hermes-interfaces.h - #54569

Closed
lavenzg wants to merge 1 commit into
react:mainfrom
lavenzg:export-D87200512
Closed

lavenzg wants to merge 1 commit into
react:mainfrom
lavenzg:export-D87200512

Conversation

@lavenzg

@lavenzg lavenzg commented Nov 17, 2025

Copy link
Copy Markdown
Contributor

Differential Revision: D87200512

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 17, 2025
@meta-codesync

meta-codesync Bot commented Nov 17, 2025

Copy link
Copy Markdown

@lavenzg has exported this pull request. If you are a Meta employee, you can view the originating Diff in D87200512.

@react-native-bot

react-native-bot commented Nov 17, 2025 •

Copy link
Copy Markdown
Collaborator
Fails
🚫

❔ This pull request needs a description.

Generated by 🚫 dangerJS against 673892a

lavenzg added a commit to lavenzg/hermes that referenced this pull request Nov 17, 2025
Summary:
Pull Request resolved: facebook#1843

X-link: react/react-native#54569

Differential Revision: D87200512
lavenzg added a commit to lavenzg/hermes that referenced this pull request Nov 19, 2025
Summary:
Pull Request resolved: facebook#1844

Pull Request resolved: facebook#1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Differential Revision: D87200512
lavenzg added a commit to lavenzg/hermes that referenced this pull request Nov 19, 2025
Summary:
Pull Request resolved: facebook#1844

Pull Request resolved: facebook#1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Differential Revision: D87200512
lavenzg added a commit to lavenzg/hermes that referenced this pull request Dec 2, 2025
Summary:
Pull Request resolved: facebook#1844

Pull Request resolved: facebook#1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Reviewed By: tmikov

Differential Revision: D87200512
lavenzg added a commit to lavenzg/hermes that referenced this pull request Dec 2, 2025
Summary:
Pull Request resolved: facebook#1844

Pull Request resolved: facebook#1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Reviewed By: tmikov

Differential Revision: D87200512
Summary:
X-link: facebook/hermes#1844

X-link: facebook/hermes#1843

Pull Request resolved: #54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Reviewed By: tmikov

Differential Revision: D87200512
meta-codesync Bot pushed a commit to facebook/hermes that referenced this pull request Dec 3, 2025
Summary:
Pull Request resolved: #1844

Pull Request resolved: #1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Reviewed By: tmikov

Differential Revision: D87200512

fbshipit-source-id: e61251f791788c8d6787ee29762c4aa76d99b68a
meta-codesync Bot pushed a commit to facebook/hermes that referenced this pull request Dec 3, 2025
Summary:
Pull Request resolved: #1844

Pull Request resolved: #1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Reviewed By: tmikov

Differential Revision: D87200512

fbshipit-source-id: e61251f791788c8d6787ee29762c4aa76d99b68a
@meta-codesync meta-codesync Bot closed this in 4bb24e2 Dec 3, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Dec 3, 2025
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @lavenzg in 4bb24e2

When will my fix make it into a release? | How to file a pick request?

@meta-codesync

meta-codesync Bot commented Dec 3, 2025

Copy link
Copy Markdown

This pull request has been merged in 4bb24e2.

cipolleschi pushed a commit to facebook/hermes that referenced this pull request Jan 14, 2026
Summary:
Pull Request resolved: #1844

Pull Request resolved: #1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Reviewed By: tmikov

Differential Revision: D87200512

fbshipit-source-id: e61251f791788c8d6787ee29762c4aa76d99b68a
cipolleschi pushed a commit to facebook/hermes that referenced this pull request Jan 14, 2026
Summary:
Pull Request resolved: #1844

Pull Request resolved: #1843

X-link: react/react-native#54569

Changelog: [Internal]

Two "hermes.h" under different prefixes (jsi/hermes.h and
hermes/hermes.h) are a bit confusing. Let's rename the JSI
header to "hermes-interfaces.h".

Reviewed By: tmikov

Differential Revision: D87200512

fbshipit-source-id: e61251f791788c8d6787ee29762c4aa76d99b68a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants