-
Notifications
You must be signed in to change notification settings - Fork 118
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
Pavel.sergeev/set skia m116 #808
Conversation
5d38bf4
to
010cad7
Compare
#809 - was merged to master, so this PR should pass all tests after a rebase. |
1f050e4
to
e870fcc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks!
Can we merge it not now, but in 1-2 weeks? Compose will need a new version of Skiko without new Skia soon (because of the release)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks!
Can we merge it not now, but in 1-2 weeks? Compose will need a new version of Skiko without new Skia soon (because of the release)
I think we can postpone, though can we publish the brunch version somewhere? I would like to try to experiment with new apis in Fleet |
@igordmn can we merge it now? |
Yes, let's merge |
e870fcc
to
f7dcf63
Compare
* update bindings according to skia-m116 * we don't have `ubuntu` suffix in skia artifacts anymore * fix compilation on windows * ignore the test because it was failing before * fix windows compilation * avgCharWidth is 0f on windows (cherry picked from commit 4217f64)
No description provided.