Skip to content

Commit 1264de3

Browse files
fix: Increase account charecter limit (#260)
I faced an issue where a long queue name is prepended with a +, which causes the test_account to fail. I recommend increasing the charecter. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Improved validation of SLURM accounts by updating the formatting of the account check command to ensure more consistent and reliable results. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent bbbcec4 commit 1264de3

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
@@ -653,7 +653,7 @@ def test_account(self, account):
653653
"""
654654
tests whether the given account is registered, raises an error, if not
655655
"""
656-
cmd = "sshare -U --format Account --noheader"
656+
cmd = "sshare -U --format Account%256 --noheader"
657657
try:
658658
accounts = subprocess.check_output(
659659
cmd, shell=True, text=True, stderr=subprocess.PIPE

0 commit comments

Comments
 (0)