googlefonts
fix: Wrong escape sequences
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
![google-cla[bot] avatar](https://avatars.githubusercontent.com/in/42202?v=4 "Published by a pub.dev verified publisher"){kind=small}
@m4rc1e Sorry it took so long for me to see your request to try this. I don’t face this issue all the time, and often, it just requires me re-running the build one or two times, and then it works. So, that’s odd, but seems to work.
However, I did test this branch, and it worked the first time, and a second time. The code changes also appear sensible enough to me, FWIW.
I fixed the merge conflict and would be happy if this gets merged, as this also fixes a SyntaxWarning since Python 3.12:
/usr/lib/python3.13/site-packages/gftools/push/items.py:220: SyntaxWarning: invalid escape sequence '\s'
return re.sub("\s+", " ", text).strip()
As the PR did: https://github.com/googlefonts/gftools/pull/1073
Or should I split this PR up?