Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 0364ccfeadd4c88445092cbed8be76d7c0315c52
      
https://github.com/tianocore/edk2/commit/0364ccfeadd4c88445092cbed8be76d7c0315c52
  Author: Ard Biesheuvel <a...@kernel.org>
  Date:   2023-06-02 (Fri, 02 Jun 2023)

  Changed paths:
    M ArmPkg/ArmPkg.ci.yaml

  Log Message:
  -----------
  ArmPkg: Enable AuditMode for Uncrustify CI checks

Uncrustify checks are too rigid, making them counter-productive:

- it leads to code that is arguably harder to parse visually (e.g.,
  the changes to ArmPkg/Include/Chipset/AArch64Mmu.h in commit
  429309e0c6b74792)
- it forces indentation-only changes to code in the vicinity of actual
  changes, making the code history more bloated than necessary (see
  commit 7f198321eec0f520373 for an example)
- finding out from the web UI what exactly Uncrustify objected to is not
  straight-forward.

So let's enable AuditMode for ArmPkg, so that interested parties can see
the uncrustify recommendations if desired, but without preventing the
changes from being merged. This leaves it at the discretion of the
ArmPkg maintainers to decide which level of conformance is required.

Cc: Leif Lindholm <quic_llind...@quicinc.com>
Cc: "Kinney, Michael D" <michael.d.kin...@intel.com>
Cc: Michael Kubacki <mikub...@linux.microsoft.com>
Signed-off-by: Ard Biesheuvel <a...@kernel.org>
Reviewed-by: Leif Lindholm <quic_llind...@quicinc.com>




_______________________________________________
edk2-commits mailing list
edk2-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to