Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35412 )
Change subject: editorconfig: add max. line length ......................................................................
Patch Set 2: Code-Review-2
(1 comment)
I strongly disagree with adding this. If it were a ruler, i'd totally support it, but it's forced wrapping, which I don't think is something we want, even at 96 characters.
This would wrap URLs in the .MD files, for example. It's just not elegant, and I don't think it was thought through.
If you want to add a max_line_length to the file, I'd set it to off.
https://review.coreboot.org/c/coreboot/+/35412/2/.editorconfig File .editorconfig:
https://review.coreboot.org/c/coreboot/+/35412/2/.editorconfig@12 PS2, Line 12: max_line_length = 96 This FORCES the maximum length (for editors that support it), which is why I didn't add it.
See this discussion: https://github.com/editorconfig/editorconfig/issues/387