Merge pull request #10 from Boehringer-Ingelheim/early_error_feedback

Early error feedback

.github/workflows/ci.yml

@@ -11,9 +11,6 @@ on:
     branches:
       - main
   push:
-    branches:
-      - main
-      - dev
   workflow_dispatch:
 
 jobs:

.lintr.R

@@ -0,0 +1,21 @@
+linters <- lintr::default_linters             # -[ diff with dv.templates 3ca8d7a10cfc7ad2307644dcac603e1f1f0feb72]-
+linters <- lintr::modify_defaults(
+  linters
+  , line_length_linter = NULL                 # we see how long lines are when we write them
+  , indentation_linter = NULL
+  , trailing_whitespace_linter = NULL
+  , cyclocomp_linter = NULL                   # prevents trivial amount of nesting and long but straightforward functions
+  , object_name_linter = NULL                 # we have reasons to capitalize. nobody in our team CamelCase. shiny does
+  , object_length_linter = NULL               # we don't type long var names just because
+  , pipe_continuation_linter = NULL           # wickham being overly prescriptive
+  , trailing_blank_lines_linter = NULL        # natural extension of trailing_whitespace_linter, present on the template
+)
+
+if(identical(Sys.getenv('CI'), "true")){
+  linters <- lintr::modify_defaults(
+    linters
+    , object_usage_linter = NULL              # R lacks var declarations; it's easy to assign to the wrong variable by mistake
+  )                                           # We only disable this lint rule on github because it fails there because
+}                                             # of a long-standing lintr bug 
+
+exclusions <- list("tests")

DESCRIPTION

@@ -1,7 +1,7 @@
 Package: dv.tables
 Type: Package
 Title: Table Modules
-Version: 0.0.1
+Version: 0.0.2
 Authors@R: c(
         person("Boehringer-Ingelheim Pharma GmbH & Co.KG", role = c("cph", "fnd")),
         person(given = "Luis", family = "Moris Fernandez", role = c("aut", "cre"), email = "[email protected]")
@@ -12,7 +12,7 @@ Encoding: UTF-8
 LazyData: true
 Roxygen: list(markdown = TRUE)
 RoxygenNote: 7.3.1
-Suggests: dv.manager (>= 1.0.9.129), jsonlite, rmarkdown, testthat (>=
+Suggests: dv.manager (>= 2.1.4), jsonlite, rmarkdown, testthat (>=
         3.0.0), shinytest2, devtools, knitr, tibble, utils
 Config/testthat/edition: 3
 Config/testthat/parallel: false
@@ -22,4 +22,4 @@ Imports: shiny (>= 1.7.1),dplyr (>= 1.0.7), purrr (>= 0.3.4),
         stats, pharmaverseadam
 Depends: R (>= 4.0)
 VignetteBuilder: knitr
-Remotes: boehringer-ingelheim/[email protected].2
+Remotes: boehringer-ingelheim/[email protected].4

NEWS.md

@@ -1,3 +1,9 @@
+# dv.tables 0.0.2
+
+* Solves minor documentation issues.
+* Removes support for data dispatchers.
+* Provides early feedback for module misconfiguration.
+
 # dv.tables 0.0.1
 
 * hierarchical_count_table.