editorconfig: bump line length to 100
Checks
Commit Message
Since we allow line lengths of up to 100, adjust the editorconfig to use
that value, to save editors (such as 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.
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
.editorconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Comments
On 10/20/21 4:48 PM, Bruce Richardson wrote:
> Since we allow line lengths of up to 100, adjust the editorconfig to use
> that value, to save editors (such as vim) from automatically wrapping
> lines at 80 characters when typing.
DPDK_CHECKPATCH_LINE_LENGTH default is 80 in
devtools/checkpatches.sh
>
> Since python checkers all seem to expect 79 character lines max, add for
> python only a 79-char max line length.
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
> .editorconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/.editorconfig b/.editorconfig
> index 5101630c8c..ab41c95085 100644
> --- a/.editorconfig
> +++ b/.editorconfig
> @@ -11,11 +11,12 @@ trim_trailing_whitespace = true
> charset = utf-8
> indent_style = tab
> tab_width = 8
> -max_line_length = 80
> +max_line_length = 100
>
> [*.py]
> indent_style = space
> indent_size = 4
> +max_line_length = 79
>
> [meson.build]
> indent_style = space
>
On Wed, Oct 20, 2021 at 04:57:41PM +0300, Andrew Rybchenko wrote:
> On 10/20/21 4:48 PM, Bruce Richardson wrote:
> > Since we allow line lengths of up to 100, adjust the editorconfig to use
> > that value, to save editors (such as vim) from automatically wrapping
> > lines at 80 characters when typing.
>
> DPDK_CHECKPATCH_LINE_LENGTH default is 80 in
> devtools/checkpatches.sh
>
Yes, but all the CIs run with it set to 100.
On Wed, Oct 20, 2021 at 04:57:41PM +0300, Andrew Rybchenko wrote:
> On 10/20/21 4:48 PM, Bruce Richardson wrote:
> > Since we allow line lengths of up to 100, adjust the editorconfig to use
> > that value, to save editors (such as vim) from automatically wrapping
> > lines at 80 characters when typing.
>
> DPDK_CHECKPATCH_LINE_LENGTH default is 80 in
> devtools/checkpatches.sh
>
I'll do a v2 with that changed in it too, just to keep things consistent.
It better reflects the state of reality in the code base.
/Bruce
@@ -11,11 +11,12 @@ trim_trailing_whitespace = true
charset = utf-8
indent_style = tab
tab_width = 8
-max_line_length = 80
+max_line_length = 100
[*.py]
indent_style = space
indent_size = 4
+max_line_length = 79
[meson.build]
indent_style = space