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
privData needs to padded with bytes not str #3129
Conversation
How did this ever work?! |
ouch. thanks looks like this wasn't covered by tests. Mind adding a simple one somewhere? |
it didn't 😅 |
@anthrotype in the middle of trying to get out a font release, tbh. Will try to come back and make a test later this week |
@anthrotype where would be best to put a test? I can see either putting it in |
Where it's more convenient to you, any test is better than nothing thanks |
@anthrotype ok, test added |
Ok to merge? |
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.
LGTM thanks!
One letter, but an important one.