fix: init_seconds_before_status_checks must be a positive integer - adjusted help message and validator

Author: cmeesters

snakemake_executor_plugin_slurm/__init__.py

@@ -154,7 +154,8 @@ class ExecutorSettings(ExecutorSettingsBase):
         default=40,
         metadata={
             "help": "Defines the time in seconds before the first status "
-            "check is performed on submitted jobs.",
+            "check is performed on submitted jobs. Must be a positive "
+            "integer",
         },
     )
 

snakemake_executor_plugin_slurm/validation.py

@@ -132,7 +132,7 @@ def validate_executor_settings(settings, logger=None):
     if settings.init_seconds_before_status_checks is not None:
         if (
             not isinstance(settings.init_seconds_before_status_checks, int)
-            or settings.init_seconds_before_status_checks < 0
+            or settings.init_seconds_before_status_checks > 0
         ):
             raise WorkflowError(
                 "init-seconds-before-status-checks must be a positive integer."