Make R CMD check pass (#34)

.codegen.json

@@ -14,6 +14,9 @@
       "Rscript -e \"devtools::install_dev_deps('.')\""
     ],
     "post_generate": [
+      "gsed -i -- 's/\\[Deprecated, use run_id instead\\]/Deprecated, use run_id instead./g' **/*.R",
+      "gsed -i -- 's/\\[MLflow endpoint\\]/MLflow endpoint/g' **/*.R",
+      "gsed -i -- \"s/\\['timestamp asc'\\]/timestamp asc/g\" **/*.R",
       "Rscript -e \"devtools::document()\""
     ]
   }

.codegen/service.R.tmpl

@@ -3,16 +3,21 @@
 #' @importFrom stats runif
 NULL
 
-{{range .Methods}}{{.Comment "#' " 80}}{{if .Request}}
+{{range .Methods}}{{.Comment "#' " 80}}
+#' @param client Required. Instance of DatabricksClient()
+{{if .Request}}
 {{if and .Wait (not .IsCrudRead)}}#'
 #' @description
 #' This is a long-running operation, which blocks until {{.Service.TitleName}} on Databricks reach
 #' {{range $i, $s := .Wait.Success}}{{if $i}} or {{end}}{{.Content}}{{end}} state with the timeout of {{.Wait.Timeout}} minutes, that you can change via `timeout` parameter.
 #' By default, the state of Databricks {{.Service.TitleName}} is reported to console. You can change this behavior
 #' by changing the `callback` parameter.
-{{end}}#' @param client Required. Instance of DatabricksClient()
-#'{{range .Request.Fields}}
-#' @param {{.SnakeName}} {{if .Required}}Required. {{end}}{{.Summary}}{{end}}
+#' @param timeout Time to wait for the operation to complete in minutes.
+#' @param callback Function to report the status of the operation. By default, it reports to console.
+{{end}}
+{{if .Request.RequiredFields}}#'
+{{end}}#'{{range .Request.Fields}}
+#' @param {{.SnakeName}} {{if .Required}}Required. {{end}}{{with .Summary}}{{.}}{{else}}This field has no description yet.{{end}}{{end}}
 {{end}}{{if .Pagination}}#'
 #' @return `data.frame` with all of the response pages.
 {{end}}#'

Makefile

@@ -4,11 +4,6 @@ deps:
 	@echo "✅ Installing dependencies"
 	@Rscript -e "if (!require(devtools)) install.packages('devtools', repos = 'https://cran.rstudio.com')"
 	@Rscript -e "devtools::install_dev_deps('.')"
-
-gen: # invoked by Databricks employees
-	@echo "✅ Regenerating files from OpenAPI"
-	@oac
-	@Rscript -e "devtools::document()"
 
 tarball:
 	@echo "✅ Building tarball"
@@ -20,7 +15,7 @@ test:
 fmt:
 	@Rscript .codegen/format.R
 
-check: gen
+check: 
 	@echo "✅ run R CMD check on packages"
 	@set -e ; \
 	cleanup () { rm *.tar.gz; rm -fr *.Rcheck; } ; \