Skip to content

Commit

Permalink
Linter cleanup @refactor (#212)
Browse files Browse the repository at this point in the history
![image](https://github.com/insightsengineering/teal.data/assets/211358/22fdcbcb-8d0c-472a-8639-9c56690f7ece)

### Changes description

- Cleans up unnecessary `# nolint` comments and adds specificity to
existing ones throughout the package

---------

Co-authored-by: Aleksander Chlebowski <[email protected]>
  • Loading branch information
averissimo and Aleksander Chlebowski authored Jan 5, 2024
1 parent 052480a commit aa47d66
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 13 deletions.
4 changes: 2 additions & 2 deletions .lintr
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
linters: linters_with_defaults(
line_length_linter = line_length_linter(120),
cyclocomp_linter = NULL,
object_usage_linter = NULL
object_usage_linter = NULL,
cyclocomp_linter = NULL
)
2 changes: 1 addition & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

* Specified minimal version of package dependencies.
* Upgraded `teal.code` dependency from `Imports` to `Depends`.
* Deprecated `get_labels` function and removed the supporting function `data_labels`.
* Deprecated `get_labels` function and removed the supporting function `data_label`.

# teal.data 0.3.0

Expand Down
2 changes: 1 addition & 1 deletion R/zzz.R
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
.onLoad <- function(libname, pkgname) { # nolint
.onLoad <- function(libname, pkgname) {
# expose default CDISC dataset names
# copy from excel file
default_cdisc_keys <- yaml::yaml.load_file(
Expand Down
18 changes: 9 additions & 9 deletions data-raw/data.R
Original file line number Diff line number Diff line change
@@ -1,28 +1,28 @@
## code to prepare `data` for testing examples
library(scda)
rADAE <- synthetic_cdisc_data("latest")$adae # nolint
rADAE <- synthetic_cdisc_data("latest")$adae # nolint: object_name_linter.
usethis::use_data(rADAE)

rADCM <- synthetic_cdisc_data("latest")$adcm # nolint
rADCM <- synthetic_cdisc_data("latest")$adcm # nolint: object_name_linter.
usethis::use_data(rADCM)

rADEX <- synthetic_cdisc_data("latest")$adex # nolint
rADEX <- synthetic_cdisc_data("latest")$adex # nolint: object_name_linter.
usethis::use_data(rADEX)

rADLB <- synthetic_cdisc_data("latest")$adlb # nolint
rADLB <- synthetic_cdisc_data("latest")$adlb # nolint: object_name_linter.
usethis::use_data(rADLB)

rADRS <- synthetic_cdisc_data("latest")$adrs # nolint
rADRS <- synthetic_cdisc_data("latest")$adrs # nolint: object_name_linter.
usethis::use_data(rADRS)

rADSL <- synthetic_cdisc_data("latest")$adsl # nolint
rADSL <- synthetic_cdisc_data("latest")$adsl # nolint: object_name_linter.
usethis::use_data(rADSL)

rADTR <- synthetic_cdisc_data("latest")$adtr # nolint
rADTR <- synthetic_cdisc_data("latest")$adtr # nolint: object_name_linter.
usethis::use_data(rADTR)

rADTTE <- synthetic_cdisc_data("latest")$adtte # nolint
rADTTE <- synthetic_cdisc_data("latest")$adtte # nolint: object_name_linter.
usethis::use_data(rADTTE)

rADVS <- synthetic_cdisc_data("latest")$advs # nolint
rADVS <- synthetic_cdisc_data("latest")$advs # nolint: object_name_linter.
usethis::use_data(rADVS)

0 comments on commit aa47d66

Please sign in to comment.