fix findstates (#733)

* fix findstates * Update src/main/resources/io/viash/runners/nextflow/states/findStates.nf Co-authored-by: Hendrik Cannoodt <[email protected]> * Update src/main/resources/io/viash/runners/nextflow/states/findStates.nf Co-authored-by: Hendrik Cannoodt <[email protected]> --------- Co-authored-by: Hendrik Cannoodt <[email protected]>
viash-io · Jul 10, 2024 · a5f3a51 · a5f3a51
1 parent 0244919
commit a5f3a51
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,8 @@ TODO add summary
   When the argument is used, a deprecation warning message is printed on stderr.
   Cannot be used together with `--engine` or `--runner` and/or `--apply_engine` or `--apply_runner`.
 
+* `nextflow_runner`: Fix refactoring error in the `findStates()` helper function (PR #733).
+
 * `viash ns exec`: Fix "relative fields" outputting absolute paths (PR# 737). Additionally, improve path resolution when using the `--src` argument.
 
 # Viash 0.9.0-RC6 (2024-06-17): Hotfix for docker image name generation

diff --git a/src/main/resources/io/viash/runners/nextflow/states/findStates.nf b/src/main/resources/io/viash/runners/nextflow/states/findStates.nf
@@ -82,8 +82,8 @@ def findStates(Map params, Map config) {
           // construct renameMap
           if (args.rename_keys) {
             def renameMap = args.rename_keys.collectEntries{renameString ->
-              def split = renameString.split(";")
-              assert split.size() == 2: "Argument 'rename_keys' should be of the form 'newKey:oldKey,newKey:oldKey'"
+              def split = renameString.split(":")
+              assert split.size() == 2: "Argument 'rename_keys' should be of the form 'newKey:oldKey', or 'newKey:oldKey;newKey:oldKey' in case of multiple values"
               split
             }