From ba814f421302e3545cfee342f508f01d0ae98ce0 Mon Sep 17 00:00:00 2001 From: Brennan Kinney <5098581+polarathene@users.noreply.github.com> Date: Sun, 26 Nov 2023 01:34:00 +1300 Subject: [PATCH] ci: Linter EC should use `/check` as the mount path (#3655) --- test/linting/lint.sh | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/test/linting/lint.sh b/test/linting/lint.sh index 7c281e8f..30cab39f 100755 --- a/test/linting/lint.sh +++ b/test/linting/lint.sh @@ -17,11 +17,14 @@ SHELLCHECK_VERSION='0.9.0' source "${REPOSITORY_ROOT}/target/scripts/helpers/log.sh" function _eclint() { + # `/check` is used instead of `/ci` as the mount path due to: + # https://github.com/editorconfig-checker/editorconfig-checker/issues/268#issuecomment-1826200253 + # `.ecrc.json` continues to explicitly ignores the `.git/` path to avoid any potential confusion if docker run --rm --tty \ - --volume "${REPOSITORY_ROOT}:/ci:ro" \ - --workdir "/ci" \ + --volume "${REPOSITORY_ROOT}:/check:ro" \ + --workdir "/check" \ --name dms-test_eclint \ - "mstruebing/editorconfig-checker:${ECLINT_VERSION}" ec -config "/ci/test/linting/.ecrc.json" + "mstruebing/editorconfig-checker:${ECLINT_VERSION}" ec -config "/check/test/linting/.ecrc.json" then _log 'info' 'ECLint succeeded' else