mbox series

[v4,0/2] standardize devtools check scripts

Message ID 20200623092935.38459-1-ciara.power@intel.com (mailing list archive)
Headers
Series standardize devtools check scripts |

Message

Power, Ciara June 23, 2020, 9:29 a.m. UTC 
  v4:
  - Merge doc patch into patch with code changes.
  - Simplified and reduced documentation and comments.

v3:
  - Fix comments on v2.
  - Add patch to update contributor's guide.

v2: Fix comments on v1.

This patchset standardizes the checkpatches and check-git-log scripts
to accept the same syntax commandline arguments, to make them easier
to use.

The output is also standardized, check-git-log previously showed no
output to the user when no errors were found, but now prints a similar
output to that of the checkpatches script.

Ciara Power (2):
  devtools: standardize script arguments
  devtools: added stats print

 devtools/check-git-log.sh           | 84 ++++++++++++++++++++---------
 devtools/checkpatches.sh            |  2 +-
 doc/guides/contributing/patches.rst | 15 ++----
 3 files changed, 64 insertions(+), 37 deletions(-)

Comments

Thomas Monjalon July 30, 2020, 10:52 p.m. UTC | #1 
23/06/2020 11:29, Ciara Power:
> v4:
>   - Merge doc patch into patch with code changes.
>   - Simplified and reduced documentation and comments.
> 
> v3:
>   - Fix comments on v2.
>   - Add patch to update contributor's guide.
> 
> v2: Fix comments on v1.
> 
> This patchset standardizes the checkpatches and check-git-log scripts
> to accept the same syntax commandline arguments, to make them easier
> to use.
> 
> The output is also standardized, check-git-log previously showed no
> output to the user when no errors were found, but now prints a similar
> output to that of the checkpatches script.
> 
> Ciara Power (2):
>   devtools: standardize script arguments
>   devtools: added stats print

Reviewed-by: Thomas Monjalon <thomas@monjalon.net>

Applied, thanks for your patience