HopkinsIDD / flepiMoP

The Flexible Epidemic Modeling Pipeline
https://flepimop.org
GNU General Public License v3.0
9 stars 4 forks source link

[Feature request]: Add `lintr` Linting For R Packages #399

Open TimothyWillard opened 1 week ago

TimothyWillard commented 1 week ago

Label

enhancement, meta/workflow, r-flepicommon, r-flepiconfig, r-inference

Priority Label

low priority

Is your feature request related to a problem? Please describe.

To improve developer productivity it would be helpful to add R linting for flepicommon, flepiconfig, and inference. This would:

  1. Minimize the number of comments/discussions in PR that relate to R code formatting, and
  2. Act as an early warning for code smells or potentially problematic code.

Successor to GH-279.

Is your feature request related to a new application, scenario round, pathogen? Please describe.

n/a

Describe the solution you'd like

The defacto standard for linting R code is lintr. Might be wise to implement this on a per package basis, pick the smallest one first, work through the issues there and then move on to the others.

pearsonca commented 1 week ago

mild extension: should also be applying lintr to any side scripts outside the packages.