Skip to content

fix: add support for react-native-windows 0.64#296

Merged
tido64 merged 1 commit intomasterfrom
tido/support-rnw-64
Mar 30, 2021
Merged

fix: add support for react-native-windows 0.64#296
tido64 merged 1 commit intomasterfrom
tido/support-rnw-64

Conversation

@tido64
Copy link
Copy Markdown
Member

@tido64 tido64 commented Mar 26, 2021

Description

Add support for react-native-windows 0.64.

Platforms affected

  • Android
  • iOS
  • macOS
  • Windows

Test plan

  1. Install react-native-windows 0.64
    yarn set-react-version 0.64
    cd example
    yarn
    yarn install-windows-test-app
    start windows/Example.sln
    yarn start:windows
    
  2. Build and run Example app

image

Comment thread example/react-native.config.js Outdated
Comment thread windows/test-app.js Outdated
Comment thread windows/test-app.js Outdated
@tido64 tido64 force-pushed the tido/support-rnw-64 branch 3 times, most recently from 918f113 to bbd31bb Compare March 27, 2021 10:18
@tido64 tido64 marked this pull request as ready for review March 27, 2021 10:56
@tido64 tido64 enabled auto-merge (squash) March 27, 2021 14:03
@tido64 tido64 force-pushed the tido/support-rnw-64 branch from bbd31bb to 30ec7f8 Compare March 30, 2021 11:10
Comment thread example/package.json
"react": "16.13.1",
"react-native": "0.63.4",
"react-native-macos": "0.63.3",
"react-native-macos": "0.63.23",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Was there a reason to update the react-native-macos version with this change?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No, I was mostly making sure that latest 0.63 still works.

// @ts-ignore intentional use of unsupported platform
const params = mockParams({ platforms: ["nextstep"], flatten: true });
expect(() => reactNativeConfig(params)).toThrowError(
expect(() => reactNativeConfig(params)).toThrow(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Was toThrowError just an alias of toThrow?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

toThrowError is apparently an old, deprecated name. It showed up under build annotations:
image

@tido64 tido64 merged commit 082040f into master Mar 30, 2021
@tido64 tido64 deleted the tido/support-rnw-64 branch March 30, 2021 17:22
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 0.5.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants