rust-vmm / rust-vmm-ci

Apache License 2.0
18 stars 33 forks source link

test_commit_format: no longer check body length #130

Closed Ablu closed 1 year ago

Ablu commented 1 year ago

There are quite a few, genuine useful cases where one wants to overflow the recommended 75 (some use 72) character rule.

Breaking long lines or manually breaking long compiler or other tool-generated outputs is just confusing and error-prone.

This does not mean that we should stop wrapping manually written text at a sane length, but enforcing the rule with a script seems to yield questionable results while creating quite a bit of friction.

Requirements

Before submitting your PR, please make sure you addressed the following requirements: