From dd97a9d1fa5010b2526feb54e90db79fc6390c1b Mon Sep 17 00:00:00 2001 From: Darshan Sen Date: Wed, 20 Apr 2022 04:56:50 +0530 Subject: [PATCH] build: improve the format-cpp error message This change specifies the target branch name instead of leaving that to the committer to fill in. Signed-off-by: Darshan Sen PR-URL: https://github.com/nodejs/node/pull/42765 Reviewed-By: Rich Trott Reviewed-By: Mestery Reviewed-By: Jiawen Geng Reviewed-By: Luigi Pinca --- .github/workflows/linters.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/linters.yml b/.github/workflows/linters.yml index 312ba0b3c9706a..fa761f4f9de044 100644 --- a/.github/workflows/linters.yml +++ b/.github/workflows/linters.yml @@ -78,7 +78,7 @@ jobs: echo echo 'ERROR: Please run:' echo - echo ' CLANG_FORMAT_START="$(git merge-base HEAD )" make format-cpp' + echo " CLANG_FORMAT_START="$\(git merge-base HEAD ${GITHUB_BASE_REF}\)" make format-cpp" echo echo 'to format the commits in your branch.' exit "$EXIT_CODE"