-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
Move 2.6.347 to release and add new release #12851
Comments
Can we try and get PR #12726 landed first, such that it's included in the next release? |
Yes, we can do this, but it would be great to have that PR included if possible. Brendan, do you have time to finish that one? |
Unless PR #12726 is likely to be updated soon, should we perhaps do the following:
|
I'd be fine with that. @brendandahl Do you think that PR can be updated soon, or should we proceed with this release? |
I've been looking into it. I'm trying to do some benchmarks, but unfortunately our current setup doesn't really measure the font conversion. I'm going to do some pushes to mozilla's talos to see how perf is there. |
Given that PR #12726 looks to be further from landing than I hoped, given the performance regressions, let's just create a new release ASAP :-) The release notes, for the new release, should probably mention that IE 11 is now unsupported. |
Yes, I'll get this done today. |
The release is done. On NPM the |
done |
Thanks! |
It's been awhile. @timvandermeij do you have time to do a new one?
The text was updated successfully, but these errors were encountered: