Skip to content

Commit 44f2e2b

Browse files
fix: missing quotes for the slurm comment (#211)
Snakemake does not correctly escape the comment it passes to slurm. When a wildcard contains for example a `(` this will lead to errors. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit Minor internal updates have been deployed to streamline how job submission commands are assembled. These refinements maintain the current functionality and user experience without introducing any visible changes. - **Refactor** - Enhanced internal handling of command formatting during job submissions to ensure consistency. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 17d488b commit 44f2e2b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

snakemake_executor_plugin_slurm/__init__.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ def run_job(self, job: JobExecutorInterface):
208208
f"--job-name {self.run_uuid} "
209209
f"--output '{slurm_logfile}' "
210210
f"--export=ALL "
211-
f"--comment {comment_str}"
211+
f"--comment '{comment_str}'"
212212
)
213213

214214
call += self.get_account_arg(job)

0 commit comments

Comments
 (0)