Skip to content

Commit

Permalink
Merge pull request #18 from PondiB/develop
Browse files Browse the repository at this point in the history
replace rlang deprecated invoke with exec keyword
  • Loading branch information
PondiB authored Apr 28, 2023
2 parents aa8158f + 13d79cc commit 82fccea
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 9 deletions.
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM --platform=linux/amd64 ubuntu:bionic
FROM ubuntu:bionic

RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y software-properties-common cmake g++ git supervisor wget
ENV TZ=Etc/UTC
Expand Down
8 changes: 4 additions & 4 deletions R/ExecutableProcess-class.R
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#' Executable process
#'
#' @include Process-class.R
#' @importFrom rlang invoke
#' @importFrom rlang exec
#' @field job Attached job to the process
#'
#' @export
Expand All @@ -13,7 +13,7 @@ ExecutableProcess <- R6Class(

#' @description Initialize executable process
#'
#' @param id Id or name of the proces
#' @param id Id or name of the process
#' @param description Shortly what the process does
#' @param parameters Used parameters in the process
#' @param operation Function that executes the process
Expand Down Expand Up @@ -76,8 +76,8 @@ ExecutableProcess <- R6Class(

}
parameterList$job = self$job
result = invoke(self$operation, parameterList)
message("Result invoked in Executable Process")
result = exec(self$operation, !!!parameterList)
message("Result invoked using exec keyword in Executable Process")

return(result)
}
Expand Down
5 changes: 1 addition & 4 deletions R/processes.R
Original file line number Diff line number Diff line change
Expand Up @@ -834,7 +834,4 @@ save_result = Process$new(
job$setOutput(format)
return(data)
}
)



)

0 comments on commit 82fccea

Please sign in to comment.