[ovs-dev] [PATCH 2/2] checkpatch: Overwrite patch file to include provisional subject.

2023-08-04 Thread Chandan Somani
If a subject is missing, then a command like utilities/checkpatch.py -S file.patch will write in the new subject to file.patch itself. Signed-off-by: Chandan Somani --- utilities/checkpatch.py | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/utilities/checkpatch.py b

[ovs-dev] [PATCH 1/2] checkpatch: Enable subject checking on git-based patches.

2023-08-04 Thread Chandan Somani
ERRORS from being printed. Signed-off-by: Chandan Somani --- utilities/checkpatch.py | 27 +++ 1 file changed, 3 insertions(+), 24 deletions(-) diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py index 5c4aaefb3..fd44bacf1 100755 --- a/utilities/checkpatch.py

Re: [ovs-dev] [PATCH v2 3/3] checkpatch: print subject field if misspelled or missing

2023-07-11 Thread Chandan Somani
On Tue, Jul 11, 2023 at 12:38 AM Eelco Chaudron wrote: > > > On 7 Jul 2023, at 22:07, Chandan Somani wrote: > > > This narrows down spelling errors that are in the commit > > subject. In v2, it also provides a subject if the subject > > line is missing. The provisio

[ovs-dev] [PATCH v2 3/3] checkpatch: print subject field if misspelled or missing

2023-07-07 Thread Chandan Somani
This narrows down spelling errors that are in the commit subject. In v2, it also provides a subject if the subject line is missing. The provisional subject is the name of the patch file, which should provide some context about the patch. Signed-off-by: Chandan Somani --- utilities/checkpatch.py

[ovs-dev] [PATCH 2/3] checkpatch: add suggestions to the spell checker

2023-07-07 Thread Chandan Somani
This will be useful for correcting possible spelling mistakes with ease. Suggestions limited to 3 at first, but configurable in the future. Signed-off-by: Chandan Somani Acked-by: Aaron Conole --- utilities/checkpatch.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/utilities

[ovs-dev] [PATCH v2 1/3] checkpatch: reorganize flagged words using a list

2023-07-07 Thread Chandan Somani
Single out flagged words and allow for more useful details, like spelling suggestions. Fixed syntax error from v1 Signed-off-by: Chandan Somani --- utilities/checkpatch.py | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/utilities/checkpatch.py b/utilities

[ovs-dev] [PATCH 2/3] checkpatch: add suggestions to the spell checker

2023-06-29 Thread Chandan Somani
This wil lbe useful for correcting possible spelling mistakes with ease. Suggestions limited to 3 at first, but configurable in the future Signed-off-by: Chandan Somani --- utilities/checkpatch.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/utilities/checkpatch.py b/utilities

[ovs-dev] [PATCH 3/3] checkpatch: print subject field if spelling error

2023-06-29 Thread Chandan Somani
This will help narrow down spelling errors that are in the commit subject Signed-off-by: Chandan Somani --- utilities/checkpatch.py | 4 1 file changed, 4 insertions(+) diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py index 57f7baf46..c5575ba00 100755 --- a/utilities

[ovs-dev] [PATCH 1/3] checkpatch: reorganize flagged words using a list

2023-06-29 Thread Chandan Somani
A list approach lets us single out each flagged word and provide more useful details, like spelling suggestions. This will be used in an upcoming patch. Signed-off-by: Chandan Somani --- utilities/checkpatch.py | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a