Open felipesanches opened 1 month ago
I agree, in principle, but I fear clarity of rationale for each check validation may be lost.
There are some checks that really validate the same kind of thing in multiple places, with the exact same rationale. For those it would be fine to clump everything in a single check.
But there are also small checks that have different reasoning each, such as the many aspects that we validate on different fields of the METADATA.pb file. Clumping those together would make it more complicated to convey to users what's the reasoning for each specific thing being checked. And it would also result in multiple "proposed" links to issue tracker entries.
I suggest we start with one initial proposal of clumping together a given set of checks, and we see how we feel about it. If we like the result, we can move on to refactor additional sets of checks.
@simoncozens, which ones would you like to propose clumping together at first?
Description/article. It's a GF-only thing so it only affects our process, and it's not as sprawling as metadata.pb.
I'd personally like to do away with the little checks tbh. However, this isn't a hill I'm willing to die on.
I'd much prefer it is there was a single check for METADATA.pb that simply compares the current metadata.pb against a generated one, since we shouldn't be editing these by hand anyway (apart from the designer name, stroke etc).
@simoncozens made a suggestion at https://github.com/fonttools/fontbakery/issues/4418#issuecomment-2024676780 and then also hinted at the idea when posting https://github.com/fonttools/fontbakery/issues/4729#issue-2303588008