Merge pull request #2637 from opral/niklasbuchfink/ninja-13-detect-if…

…-translation-files-have-changed fix: NINJA-13 no reports after project fix
opral · Apr 23, 2024 · 811f50b · 811f50b
2 parents 7f22d82 + 2144228
commit 811f50b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.changeset/mean-students-argue.md b/.changeset/mean-students-argue.md
@@ -0,0 +1,5 @@
+---
+"@inlang/github-lint-action": minor
+---
+
+fix lint report comment after a project has been fixed
diff --git a/inlang/source-code/github-lint-action/src/main.ts b/inlang/source-code/github-lint-action/src/main.ts
@@ -195,12 +195,12 @@ ${error?.cause.stack}`
 	.join("\n")}`
 				continue
 			}
-			// Case: setup of project fixed -> no comment
+			// Case: setup of project fixed -> comment with new lint reports
 			if (result.errorsBase.length > 0 && result.errorsHead.length === 0) {
-				console.debug(`#### ✅ Setup of project \`${result.projectPath}\` fixed`)
+				console.debug(`✅ Setup of project \`${result.projectPath}\` fixed`)
 			}
 			// Case: No lint reports found -> no comment
-			if (result.errorsBase.length > 0 || result.errorsHead.length > 0) continue
+			if (result.errorsHead.length > 0) continue
 			if (result.lintSummary.length === 0) continue
 			// Case: Lint reports found -> create comment with lint summary
 			const lintSummary = result.lintSummary