search

epiforecasts / scoringutils

Utilities for Scoring and Assessing Predictions
https://epiforecasts.io/scoringutils/
Other
48 stars 20 forks source link

Clean up linter statements around `cli` conditions #807

Open nikosbosse opened 5 months ago

nikosbosse commented 5 months ago

As suggested by Hugo, we should clean the linter statements surrounding cli.

The syntax to disable a specific linter is quite fussy and requires a period at the end. The current comment disables all linter. The simplest option is probably to follow what this linter is suggesting in all cases:

#nolint start: keyword_quote_linter
cli_abort(
c(
`!` = "If a column 'scale' is present, entries with scale =='natural'
are required for the transformation."
)
)

_Originally posted by @Bisaloo in https://github.com/epiforecasts/scoringutils/pull/791#discussion_r1567262476_