Lorenz Walthert
Lorenz Walthert
Thanks for all your comments. I added some suggestions to the PR, please have a look. I am preparing a new CRAN release but I suggest to not include this...
Ok, works for me too. To be honest, I don't think `@eval` is used that often anyways. We could also make pre-commit catch more problems related to roxygen code comments...
FWIW you can always override `language:` (and other fields) in your `.pre-commit-config.yaml`.
Can you please fix the one test in R cmd check that fails, then I can merge this...
Thanks, since we use `language: r`, honouring 0.7.3 can be guaranteed.
Thanks again for the collaboration. I added an issue for extending `deps-in-desc` to check roxygen code examples as well: https://github.com/lorenzwalthert/precommit/issues/585. Feel free to contribute.
@assignUser would you be ok with that transfer or do you think it should live elsewhere? I also considered r-opensci but I think the hurdles for this would be higher...
@assignUser can we merge this?
Another use case: Want to add a dependency to a project that is not installable on the developer platform like macOS, e.g. because missing binaries, built-time requirements etc. and the...
@DavisVaughan and @lionel-, I just created the formatting diff for {styler} and air on {styler}'s test cases. As discussed, this might be helpful for you to identify (edge) cases you...