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

#1665: Fixed issue with RN Skia not building on RN 0.72 #1668

Merged
merged 2 commits into from Jun 24, 2023

Conversation

chrfalch
Copy link
Collaborator

The reason is that we were using some now-deprecated setings for artifact publishing.

We've now done as Expo does here (expo/expo#22537) to remedy the error.

I've tested it with a fresh 0.72 install and also with our 0.72 example version.

The updated code was added by @mrousavy to accomplish some magic tricks in vision camera.

Thanks to @Kudo who once again keeps his eyes open for so many small things. <3

fixes #1665

The reason is that we were using some now-deprecated setings for artifact publishing.

We've now done as Expo does here (expo/expo#22537) to remedy the error.

I've tested it with a fresh 0.72 install and also with our 0.72 example version.
@chrfalch chrfalch requested a review from wcandillon June 24, 2023 11:53
@chrfalch chrfalch merged commit 05f8bfe into main Jun 24, 2023
6 checks passed
@chrfalch chrfalch deleted the fix/1665-fix-android-gradle-8-rn-072 branch June 24, 2023 14:46
@Kudo
Copy link
Contributor

Kudo commented Jun 26, 2023

it's our pleasure to have your support for expo sdk. thanks @chrfalch! will update to new react-native-skia version once it's published.

@wcandillon
Copy link
Collaborator

@Kudo it's published at 0.1.196

@Kudo
Copy link
Contributor

Kudo commented Jun 28, 2023

thanks @wcandillon! having the pr on the go

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.

Issue with Gradle 8 in React Native 72
3 participants