Skip to content

Commit 7daf301

Browse files
authored
fix: validator check for integers was broken. (#381)
<!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Validation updated for the initialization delay before status checks: integer values of 1 and above are now accepted, whereas previously only zero or negative values were allowed. Type checking remains enforced so only integers are permitted. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 4458d23 commit 7daf301

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

snakemake_executor_plugin_slurm/validation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ def validate_executor_settings(settings, logger=None):
132132
if settings.init_seconds_before_status_checks is not None:
133133
if (
134134
not isinstance(settings.init_seconds_before_status_checks, int)
135-
or settings.init_seconds_before_status_checks > 0
135+
or settings.init_seconds_before_status_checks < 1
136136
):
137137
raise WorkflowError(
138138
"init-seconds-before-status-checks must be a positive integer."

0 commit comments

Comments
 (0)