[openssl/openssl] c2083f: feat: enhance util/check-format-commit.sh to be ab...

Richard Levitte noreply at github.com
Fri Jul 12 14:39:57 UTC 2024


  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: c2083f4ec608b14b48a43c5d56798a01f1cf4931
      https://github.com/openssl/openssl/commit/c2083f4ec608b14b48a43c5d56798a01f1cf4931
  Author: Richard Levitte <levitte at openssl.org>
  Date:   2024-07-12 (Fri, 12 Jul 2024)

  Changed paths:
    M util/check-format-commit.sh

  Log Message:
  -----------
  feat: enhance util/check-format-commit.sh to be able to handle a commit range

Additionally, the 'git diff' call is modified to not show context lines, as
it's confusing to have style nits displayed on lines the author of the
commits hasn't touched.

Reviewed-by: Neil Horman <nhorman at openssl.org>
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24856)


  Commit: edb5dd56fcbeff335f2ab59e8b76780043695814
      https://github.com/openssl/openssl/commit/edb5dd56fcbeff335f2ab59e8b76780043695814
  Author: Richard Levitte <levitte at openssl.org>
  Date:   2024-07-12 (Fri, 12 Jul 2024)

  Changed paths:
    M .github/workflows/style-checks.yml

  Log Message:
  -----------
  fix(stylecheck): Take advantage of check-format-commit.sh's new capability

.github/workflows/style-checks.yml now runs util/check-format-commit.sh
with the whole range of commits of the given PR.  This allows code style
fixups to be in a separate commit.

Reviewed-by: Neil Horman <nhorman at openssl.org>
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24856)


Compare: https://github.com/openssl/openssl/compare/b74646b627ad...edb5dd56fcbe

To unsubscribe from these emails, change your notification settings at https://github.com/openssl/openssl/settings/notifications


More information about the openssl-commits mailing list