Skip to content

[pull] main from microsoft:main #33

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

pull[bot]
Copy link

@pull pull bot commented Apr 24, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

## Summary:

It's not clear (both to humans and to build tools like `nx`) that the
only packages we fork from the React Native monorepo upstream are `
'react-native' --> 'react-native-macos'` and
`'@react-native/virtualized-lists' -->
'@react-native-mac/virtualized-lists'`. The rest of the packages should
essentially only ever be used in local development, while the published
package `react-native-macos` uses the upstream versions of these
packages. We can solve this by marking the packages as private.

## Test Plan:

IIRC, this caused some build failure last time I tried this.. let's see
how CI reacts to this change. Further testing with `nx release` should
be done locally to make sure the nx build graph stops marking the
unforked monoreppo packages as dependencies.
@pull pull bot added the ⤵️ pull label Apr 24, 2025
hansenyy and others added 3 commits May 15, 2025 16:22
## Summary:

`findLastestVersionWithArtifact` fails to find older versions of Hermes
prebuilds because the Maven endpoint it uses does not return all
versions.

Resolves #2499

## Test Plan:

`pod install` output from RNTA with RN 0.78.2 and RNM 0.78.3

![image](https://github.com/user-attachments/assets/a6984ba0-87c0-4304-95d1-4a5b054b44fa)
## Summary:

`TextPropsMacOS` has been using `MouseEvent` defined by web as opposed
to React's.

## Test Plan:

n/a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants