> From: dev <dev-boun...@dpdk.org> On Behalf Of Bruce Richardson > Sent: Wednesday 20 October 2021 15:26 > To: dev@dpdk.org > Cc: Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>; Richardson, > Bruce <bruce.richard...@intel.com> > Subject: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 > characters are ok > > Since we allow line lengths of up to 100, and the CI checkpatches job > only check for that amount, the rest of our tooling and docs should > reflect this reality. Therefore we can: > > * adjust the editorconfig to use that value, to save editors (e.g. vim) > from automatically wrapping lines at 80 characters when typing. > [Since python checkers all seem to expect 79 character lines max, add > for python only a 79-char max line length.] > > * change the default line length setting in checkpatches script to 100 > so as it matches CI and pre-merge checks. > > * update the docs to clarify that while 80 chars is recommended, up to > 100 characters is acceptable. > > Signed-off-by: Bruce Richardson <bruce.richard...@intel.com> > ---
I think it would be great to have clarity around this, since it seems to already be allowed. Acked-by: Conor Walsh <conor.wa...@intel.com>
