Display warning instead of error when no files are matched, and retur…

…n with exit code 0 Closes pinterest#1624
paul-dingemans · Sep 3, 2022 · bb11a86 · bb11a86
1 parent e5ff031
commit bb11a86
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,6 +13,7 @@ Do not show deprecation warning about property "disabled_rules" when using CLi-p
 ### Added
 
 ### Changed
+* Display warning instead of error when no files are matched, and return with exit code 0. ([#1624](https://github.com/pinterest/ktlint/issues/1624))
 
 ### Removed
 

diff --git a/ktlint/src/main/kotlin/com/pinterest/ktlint/internal/KtlintCommandLine.kt b/ktlint/src/main/kotlin/com/pinterest/ktlint/internal/KtlintCommandLine.kt
@@ -274,11 +274,12 @@ internal class KtlintCommandLine {
             )
         }
         reporter.afterAll()
+        logger.debug { "${System.currentTimeMillis() - start}ms / $fileNumber file(s) / $errorNumber error(s)" }
         if (fileNumber.get() == 0) {
-            logger.error { "No files matched $patterns" }
-            exitProcess(1)
+            // Do not return an error as this would implicate that in a multi-module project, each module has to contain
+            // at least one kotlin file.
+            logger.warn { "No files matched $patterns" }
         }
-        logger.debug { "${System.currentTimeMillis() - start}ms / $fileNumber file(s) / $errorNumber error(s)" }
         if (tripped.get()) {
             exitProcess(1)
         }