-
Couldn't load subscription status.
- Fork 1.2k
Integrate 0.82.0-nightly-20250806-5936f29d6 #15261
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
base: main
Are you sure you want to change the base?
Conversation
Added to avoid conflicts in integration
c6cd45a to
ba72f25
Compare
fe9124e to
b4eabb5
Compare
5867212 to
dc01d2e
Compare
b3471d0 to
7d606cc
Compare
| new RegExp(`${rnwPath}/build/[/\\]`), | ||
| new RegExp(`${rnwPath}/target/[/\\]`), | ||
| /[/\\]\.ProjectImports\.zip/, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.* is not the same as [/\\]. The first greedy matches any character, the latter only matches a single instance / or \. We already have a trailing / here so we shouldn't add another:
| new RegExp(`${rnwPath}/build/[/\\]`), | |
| new RegExp(`${rnwPath}/target/[/\\]`), | |
| /[/\\]\.ProjectImports\.zip/, | |
| new RegExp(`${rnwPath}/build/`), | |
| new RegExp(`${rnwPath}/target/`), | |
| /\.ProjectImports\.zip$/, |
I see you've done the same mistake in the other config files as well. I suggest you revert to what you had previously as what I did in microsoft/rnx-kit#3783 was merely performance optimizations. We can go over these in a separate PR.
b6e8e20 to
967d159
Compare
eaafbdb to
0ef9f78
Compare
Description
react-native-community/cli#2491
Type of Change
Erase all that don't apply.
Why
What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits.
Resolves [Add Relevant Issue Here]
What
What changes were made to the codebase to solve the bug, add the functionality, etc. that you specified above.
Screenshots
Add any relevant screen captures here from before or after your changes.
Testing
If you added tests that prove your changes are effective or that your feature works, add a few sentences here detailing the added test scenarios.
Optional: Describe the tests that you ran locally to verify your changes.
Changelog
Should this change be included in the release notes: indicate yes or no
Add a brief summary of the change to use in the release notes for the next release.
Microsoft Reviewers: Open in CodeFlow