Closed richfitz closed 1 year ago
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:
Coverage data is based on head (
0881ac1
) compared to base (35d1847
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
The new version of lintr changes behaviour of some linters, this PR fixes the newly reported lint.
I'm not a huge fan of the requirement for braces on anonymous functions, but there's no way of selectively turning that part off. The nolint tags for the 1:x expressions are because they never get evaluated by R but get used as part of the DSL so we need them to use the 1:x form, not the seq_len(x) form