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/support skia v0.1.233 #96

Merged
merged 2 commits into from
Mar 11, 2024

Conversation

sunrise1002
Copy link
Contributor

Support skia v0.1.233

@gabimoncha
Copy link

@sunrise1002 I would decrease the version to 0.1.221 since this is the one supported by expo too.

@daxaxelrod
Copy link

daxaxelrod commented Feb 14, 2024

I guess I should have checked before I went and did the same thing haha #99

const paths = useSharedValue<{ ...

seems to be giving me a bit of trouble, getting host object errors

@gabimoncha
Copy link

@mrousavy @chrispader - I confirm this PR fixes the graph with the latest Skia version 0.1.241
The latest version will make the graph compatible with Expo and Expo web

@mrousavy
Copy link
Member

I'll take a look in a few hours!

@mrousavy
Copy link
Member

Looks good! I think we can bump even further to latest, lgtm

@mrousavy mrousavy merged commit d38c205 into margelo:main Mar 11, 2024
@dsshard
Copy link

dsshard commented Mar 12, 2024

@mrousavy Hello! Will it be possible to make a new version in npm?

@mrousavy
Copy link
Member

Sure - done!

katharine-river added a commit to katharine-river/react-native-graph that referenced this pull request Jul 30, 2024
…ch (#3)

* Update README.md

* feat: Support skia v0.1.233 (margelo#96)

* feat: Support skia v0.1.233

* feat: Support skia v0.1.233

* chore: release 1.1.0

* Lockfiles

* chore: update skia version (#4)

* chore: update skia version

* fix: fix path interpolation animation

---------

Co-authored-by: Marc Rousavy <me@mrousavy.com>
Co-authored-by: sunrise1002 <tunglm.mar@gmail.com>
Co-authored-by: katharine-river <132921466+katharine-river@users.noreply.github.com>
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.

None yet

5 participants