Skip to content

Fix component stacks for tsx, ts, and jsx files - #43370

Closed
rickhanlonii wants to merge 1 commit into
react:mainfrom
rickhanlonii:export-D54638526
Closed

Fix component stacks for tsx, ts, and jsx files#43370
rickhanlonii wants to merge 1 commit into
react:mainfrom
rickhanlonii:export-D54638526

Conversation

@rickhanlonii

Copy link
Copy Markdown
Contributor

Summary:
Component stacks with files ending in .ts, .tsx, or .jsx were skipped in LogBox reporting. This diff fixes the regex.

Changelog:
[General][Fixed] - Support .tsx, .ts, and .jsx in component stacks

Differential Revision: D54638526

Summary:
Component stacks with files ending in .ts, .tsx, or .jsx were skipped in LogBox reporting. This diff fixes the regex.
Changelog:
[General][Fixed] - Support .tsx, .ts, and .jsx in component stacks
Differential Revision: D54638526
@facebook-github-botfacebook-github-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 Mar 7, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D54638526

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D54638526

rickhanlonii added a commit to rickhanlonii/react-native that referenced this pull request Mar 7, 2024
Summary:
Component stacks with files ending in .ts, .tsx, or .jsx were skipped in LogBox reporting. This diff fixes the regex.
Changelog:
[General][Fixed] - Support .tsx, .ts, and .jsx in component stacks
Differential Revision: D54638526
@facebook-github-botfacebook-github-bot added the Merged This PR has been merged. label Mar 8, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in adaf5eb.

@Titozzz

Copy link
Copy Markdown
Collaborator

I've also requested a cherry pick for 0.73 here => reactwg/react-native-releases#133

alfonsocj pushed a commit that referenced this pull request Mar 8, 2024
Summary:
Pull Request resolved: #43370
Component stacks with files ending in .ts, .tsx, or .jsx were skipped in LogBox reporting. This diff fixes the regex.
Changelog:
[General][Fixed] - Support .tsx, .ts, and .jsx in component stacks
Reviewed By: yungsters
Differential Revision: D54638526
fbshipit-source-id: a5271daaa7b687e8e075be3f94ab9b9c03f79b66
huntie pushed a commit that referenced this pull request Mar 11, 2024
Summary:
Pull Request resolved: #43370
Component stacks with files ending in .ts, .tsx, or .jsx were skipped in LogBox reporting. This diff fixes the regex.
Changelog:
[General][Fixed] - Support .tsx, .ts, and .jsx in component stacks
Reviewed By: yungsters
Differential Revision: D54638526
fbshipit-source-id: a5271daaa7b687e8e075be3f94ab9b9c03f79b66
@cipolleschicipolleschi mentioned this pull request Jul 22, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.fb-exportedMergedThis PR has been merged.p: FacebookPartner: FacebookPartner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rickhanlonii@facebook-github-bot@Titozzz