Add validation for Pango Markup - Fixes #8

ManimCommunity / ManimPango

Binding for Pango, to use with Manim.

https://manimpango.manim.community

MIT License

43 stars 13 forks source link

Add validation for Pango Markup - Fixes #8 #9

Closed PhilippImhof closed 3 years ago

PhilippImhof commented 3 years ago

Fixes #8

PhilippImhof commented 3 years ago

As soon as this is merged, I file another PR for the change of MarkupText in Manim

naveen521kk commented 3 years ago

Also, it would be good to have a small test for that :)

PhilippImhof commented 3 years ago

Also, it would be good to have a small test for that :)

I will produce one in the next few days.

PhilippImhof commented 3 years ago

How can black fail and tell me the line no. 5 is too long? It's 44 chars now....

PhilippImhof commented 3 years ago

The lint check is driving me mad. First it complained, because the line was longer than 79 chars. And now it seemingly puts all the stuff on one line again?

What length am I supposed to use? @naveen521kk