From 5ff3a01deed538d79c462ba6da8bfa9c6f2eb939 Mon Sep 17 00:00:00 2001 From: Jacob Overgaard <752371+iOvergaard@users.noreply.github.com> Date: Wed, 3 Jul 2024 12:59:11 +0200 Subject: [PATCH] build: try and map the found files back to the src folder --- .../devops/build/check-path-length.js | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/devops/build/check-path-length.js b/src/Umbraco.Web.UI.Client/devops/build/check-path-length.js index f45a622e7e..3b48ed22b0 100644 --- a/src/Umbraco.Web.UI.Client/devops/build/check-path-length.js +++ b/src/Umbraco.Web.UI.Client/devops/build/check-path-length.js @@ -28,9 +28,9 @@ function checkPathLength(dir) { hasError = true; if (IS_AZURE_PIPELINES) { - console.error(`##vso[task.logissue type=error;sourcepath=${filePath};]Path exceeds maximum length of ${MAX_PATH_LENGTH} characters: ${filePath} with ${filePath.length} characters`); + console.error(`##vso[task.logissue type=error;sourcepath=${mapFileToSourcePath(filePath)};]Path exceeds maximum length of ${MAX_PATH_LENGTH} characters: ${filePath} with ${filePath.length} characters`); } else if (IS_GITHUB_ACTIONS) { - console.error(`::error file=${filePath},title=Path exceeds ${MAX_PATH_LENGTH} characters::Paths should not be longer than ${MAX_PATH_LENGTH} characters to support WIN32 systems. The file ${filePath} exceeds that with ${filePath.length - MAX_PATH_LENGTH} characters.`); + console.error(`::error file=${mapFileToSourcePath(filePath)},title=Path exceeds ${MAX_PATH_LENGTH} characters::Paths should not be longer than ${MAX_PATH_LENGTH} characters to support WIN32 systems. The file ${filePath} exceeds that with ${filePath.length - MAX_PATH_LENGTH} characters.`); } else { console.error(`Path exceeds maximum length of ${MAX_PATH_LENGTH} characters: ${FILE_PATH_COLOR}`, filePath, filePath.length - MAX_PATH_LENGTH); } @@ -47,6 +47,16 @@ function checkPathLength(dir) { return hasError; } +/** + * Maps a file path to a source path for CI logs. + * @remark This might not always work as expected, especially on bundled files, but it's a best effort to map the file path to a source path. + * @param {string} file - The file path to map to a source path + * @returns {string} + */ +function mapFileToSourcePath(file) { + return file.replace(PROJECT_DIR, 'src'); +} + const hasError = checkPathLength(PROJECT_DIR, MAX_PATH_LENGTH); if (hasError) {