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.

Sign up for GitHub

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

Jump to bottom

[FontBuilder] Update setupAvar() to also support avar-2, fixing _add_avar() call site #3183

Merged

justvanrossum merged 3 commits into main from issue3182

Jun 25, 2023

Collaborator

justvanrossum commented Jun 25, 2023 •

edited

Fixes #3182.

The avar-2 bit is untested, but I did add a minimal FontBuilder avar test that would have caught #3182.

justvanrossum added 3 commits

June 25, 2023 15:31


          Update setupAvar to also support avar-2, fixing _add_avar() call site

d59291a


          Fix exception message

b4ce817


          Add minimal avar test

a912f7d

Member

behdad commented Jun 25, 2023

LGTM. Sorry about the breakage.

justvanrossum merged commit 44cfe35 into main

10 checks passed

justvanrossum deleted the issue3182 branch

June 25, 2023 20:24

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