Replace pr-commands.yaml with pre-commit rules

russHyde / dupree

{dupree} helps identify code blocks that have a high level of similarity in a set of R files

https://russhyde.github.io/dupree/

Other

37 stars 0 forks source link

Replace pr-commands.yaml with pre-commit rules #83

Open russHyde opened 1 year ago

russHyde commented 1 year ago

Aim, to ensure that styler::style_pkg() and devtools::document() are ran on every commit. This should keep the code clean, and is more hands off than having PR discussion commands for styling and documenting the code.

russHyde commented 1 year ago

Note there have been some issues doing this.

The pre-commit lint rule needs to be turned off (because it flags lots of needless object-usage errors)
Dependencies in ./presentations/ should be ignored, they do not need to be in DESCRIPTION (? how to ignore this folder in "deps-in-desc" rule)
./docs, ./presentations and ./tests/testthat/testdata should not be modified (by styler etc)