Skip to content
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

feat: add Windows New Arch Support #278

Merged

Conversation

Yajur-Grover
Copy link
Contributor

Re-creating #274

Overview

Add support for the New Architecture on Windows.

Test Plan

Run the following in a command line tool at the clipboard root:

yarn
yarn start:windows:fabric

Screenshots of Example app on both Old and New Architecture:

Old Arch:
image

New Arch:
clipboard-fabric

Note: hitting some intermittent hoisting issues with the example app, where duplicate copies of certain packages are being created in both ~/package.json and ~/example/package.json, causing build issues for the example app.

@Yajur-Grover Yajur-Grover marked this pull request as ready for review January 7, 2025 23:32
@Yajur-Grover
Copy link
Contributor Author

@Naturalclar I have opened this to replace #274 without the last 0.76 commit. Let me know if this needs anything else.

Copy link
Member

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Naturalclar Naturalclar merged commit b18b626 into react-native-clipboard:master Jan 8, 2025
Comment on lines +46 to +49
"dependencies": {
"react-native": "^0.75.3",
"react-native-windows": "^0.75.0"
},

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this introduces a dependency on these for apps that install the latest version of RNClipboard, leading to duplicate versions of RN. was there any reason to move these to direct dependencies?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will be fixed in #279

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Naturalclar thank you for tackling the issue!

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

Successfully merging this pull request may close these issues.

3 participants