Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31654 )
Change subject: util/lint: Fix clang-format test and enable it by default ......................................................................
Patch Set 3:
(1 comment)
I don't think this is something that many people will be happy about either. These tools aren't perfect and sometimes you *want* to ignore style in specific cases. Looks to me like this isn't actually doing anything right now due to the lack of a scope file(?), but if you plan to add that later, it's probably going to bring trouble as well.
https://review.coreboot.org/#/c/31654/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/31654/3//COMMIT_MSG@13 PS3, Line 13: $(top)/.clang-format-scope I may be blind, but I don't see that file anywhere in the tree. Where is it? Or is this only supposed to be for out-of-tree payloads?