Skip to content

Removing all references of hz_tracing V2 from the codebase - #54062

Closed
jamezmoran wants to merge 1 commit into
react:mainfrom
jamezmoran:export-D83808657
Closed

Removing all references of hz_tracing V2 from the codebase#54062
jamezmoran wants to merge 1 commit into
react:mainfrom
jamezmoran:export-D83808657

Conversation

@jamezmoran

Copy link
Copy Markdown

Summary:
Renaming every V2 tracing macro so that it no longer has _V2 suffixed. This diff completes the migration of the hz_tracing V1->V2 API.

This is a very large diff but is comprised of mostly mechanical changes. Most of these files were edited using find and sed, specifically:
find . -type f \( -name "*.h" -o -name "*.cpp" \) -exec sed -i 's/HZT_\(.*\)_V2/HZT_\1/g' {} +.

Since this is just a rename, no difference in functionality is expected.

Differential Revision: D83808657

@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 Oct 4, 2025
@meta-codesync

meta-codesync Bot commented Oct 4, 2025

Copy link
Copy Markdown

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

Summary:
Pull Request resolved: react#54062

Renaming every V2 tracing macro so that it no longer has `_V2` suffixed. This diff completes the migration of the hz_tracing V1->V2 API.

This is a very large diff but is comprised of mostly mechanical changes. Most of these files were edited using `find` and `sed`, specifically:
`find . -type f \( -name "*.h" -o -name "*.cpp" \) -exec sed -i 's/HZT_\(.*\)_V2/HZT_\1/g' {} +`.

Since this is just a rename, no difference in functionality is expected.

Differential Revision: D83808657
@meta-codesync meta-codesync Bot closed this in c24e67f Oct 7, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Oct 7, 2025
@meta-codesync

meta-codesync Bot commented Oct 7, 2025

Copy link
Copy Markdown

This pull request has been merged in c24e67f.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by James Moran in c24e67f

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

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