From 40dcbb1483f527d919430386efb08401be8be55a Mon Sep 17 00:00:00 2001 From: Wolfgang Walther Date: Wed, 16 Jul 2025 11:59:28 +0200 Subject: [PATCH] ci/github-script/commits: ignore whitespace on diff This reduces noise that the cherry-pick reviews produce when formatting related conflicts had to be resolved. We only do this in the length-limited review comment, though. All changes, including whitespace, can still be double-checked in the job log if needed. --- ci/github-script/commits.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ci/github-script/commits.js b/ci/github-script/commits.js index f64299c49886..7a3290faba42 100644 --- a/ci/github-script/commits.js +++ b/ci/github-script/commits.js @@ -70,6 +70,7 @@ module.exports = async function ({ github, context, core }) { __dirname, 'range-diff', '--no-color', + '--ignore-all-space', '--no-notes', // 100 means "any change will be reported"; 0 means "no change will be reported" '--creation-factor=100', @@ -119,7 +120,7 @@ module.exports = async function ({ github, context, core }) { const results = await Promise.all(commits.map(handle)) - // Log all results without truncation and with better highlighting to the job log. + // Log all results without truncation, with better highlighting and all whitespace changes to the job log. results.forEach(({ sha, commit, severity, message, colored_diff }) => { core.startGroup(`Commit ${sha}`) core.info(`Author: ${commit.author.name} ${commit.author.email}`)