Skip to content

Commit

Permalink
Merge pull request #274 from viash-io/fix/issue_273
Browse files Browse the repository at this point in the history
`config inject`: Doesn't work when `must_exist == true` (#273).
  • Loading branch information
rcannood authored Oct 26, 2022
2 parents 46df237 + 3a16a71 commit 3b50337
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@

* `NextflowVDSL3Platform`: Fix 'Module scriptPath has not been defined yet' error when Nextflow>=22.10 (#269).

* `config inject`: Doesn't work when `must_exist == true` (#273).

## DEPRECATION

* `DockerRequirements`: The `resources:` setting has been deprecated and will be removed in Viash 0.7.0. Please use `copy:` instead.
Expand Down
6 changes: 6 additions & 0 deletions src/main/scala/io/viash/platforms/DebugPlatform.scala
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,12 @@ case class DebugPlatform(
arg.copy(required = false, default = One("value"), choices = Nil)
case a => a
}
// also turn off must_exist if it is set to true
.map{
case arg: FileArgument if arg.must_exist =>
arg.copy(must_exist = false)
case a => a
}
.map{Right(_)}
)
}
Expand Down

0 comments on commit 3b50337

Please sign in to comment.