[Added][CLI] --fail-on-warnings

To return error when we detected warning

See #828

Author: set-soft

CHANGELOG.md

@@ -7,6 +7,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 ### Added
+- CLI:
+  - --fail-on-warnings: to return error when we detected warning (See #828)
 - iBoM: `mark_when_checked` option (#881)
 
 

docs/source/Changelog.rst

@@ -19,6 +19,11 @@ Versioning <https://semver.org/spec/v2.0.0.html>`__.
 Added
 ~~~~~
 
+-  CLI:
+
+   -  –fail-on-warnings: to return error when we detected warning (See
+      #828)
+
 -  iBoM: ``mark_when_checked`` option (#881)
 
 `1.8.5 <https://github.com/INTI-CMNB/KiBot/compare/v1.8.4...v1.8.5>`__ - 2025-11-26

docs/source/errors.rst

@@ -42,3 +42,4 @@ Supported error levels
 - 36: WARN_AS_ERROR
 - 37: CHECK_FIELD
 - 38: IGNORED_ERRORS
+- 39: GOT_WARNINGS

docs/source/usage.txt

@@ -51,6 +51,7 @@ Options:
   -F, --fail-on-ignored            Return an error code if we skipped a fail.
                                    Used in conjunction with -D and `dont_stop`
                                    options
+  --fail-on-warnings               Return an error code if we informed warnings.
   -g DEF, --global-redef DEF       Overwrite a global value (VAR=VAL)
   --gui                            Open a graphic dialog
   --internal-check                 Run some outputs internal checks

kibot/__main__.py

@@ -58,6 +58,7 @@
   -F, --fail-on-ignored            Return an error code if we skipped a fail.
                                    Used in conjunction with -D and `dont_stop`
                                    options
+  --fail-on-warnings               Return an error code if we informed warnings.
   -g DEF, --global-redef DEF       Overwrite a global value (VAR=VAL)
   --gui                            Open a graphic dialog
   --internal-check                 Run some outputs internal checks
@@ -153,7 +154,7 @@
 from .gs import GS
 from . import dep_downloader
 from .misc import (EXIT_BAD_ARGS, W_VARCFG, NO_PCBNEW_MODULE, W_NOKIVER, hide_stderr, TRY_INSTALL_CHECK, W_ONWIN,
-                   FAILED_EXECUTE, W_ONMAC, IGNORED_ERRORS)
+                   FAILED_EXECUTE, W_ONMAC, IGNORED_ERRORS, GOT_WARNINGS)
 from .pre_base import BasePreFlight
 from .config_reader import (print_outputs_help, print_output_help, print_preflights_help, create_example, print_filters_help,
                             print_global_options_help, print_dependencies, print_variants_help, print_errors,
@@ -668,6 +669,9 @@ def main():
     if args.fail_on_ignored and (GS.errors_ignored or log.errors_ignored):
         exit(IGNORED_ERRORS)
 
+    if args.fail_on_warnings and logger.got_warnings():
+        exit(GOT_WARNINGS)
+
 
 if __name__ == "__main__":
     main()   # pragma: no cover

kibot/log.py

@@ -203,6 +203,9 @@ def log_totals(self):
                 filt_msg = ', {} filtered'.format(MyLogger.n_filtered)
             self.info('Found {} unique warning/s ({} total{})'.format(MyLogger.warn_cnt, MyLogger.warn_tcnt, filt_msg))
 
+    def got_warnings(self):
+        return MyLogger.warn_cnt or MyLogger.warn_tcnt
+
     def non_critical_error(self, msg, *args, **kwargs):
         buf = str(msg)
         push_error_msg(buf)

kibot/misc.py

@@ -52,6 +52,7 @@
 WARN_AS_ERROR = 36
 CHECK_FIELD = 37
 IGNORED_ERRORS = 38
+GOT_WARNINGS = 39   # Not treated as errors, but using `--fail-on-warnings`
 error_level_to_name = ['NONE',
                        'INTERNAL_ERROR',
                        'WRONG_ARGUMENTS',
@@ -90,7 +91,8 @@
                        'BLENDER_ERROR',
                        'WARN_AS_ERROR',
                        'CHECK_FIELD',
-                       'IGNORED_ERRORS'
+                       'IGNORED_ERRORS',
+                       'GOT_WARNINGS'
                        ]
 KICOST_SUBMODULE = '../submodules/KiCost/src/kicost'
 EXAMPLE_CFG = 'example_template.kibot.yaml'