New pipe_return_linter #2299
New pipe_return_linter #2299
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2299 +/- ##
=======================================
Coverage 99.40% 99.40%
=======================================
Files 117 117
Lines 5334 5334
=======================================
Hits 5302 5302
Misses 32 32 ☔ View full report in Codecov by Sentry. |
| expect_lint(nested_return_lines, NULL, linter) | ||
| }) | ||
|
|
||
| test_that("pipe_return_linter blocks simple disallowed usages", { |
There was a problem hiding this comment.
I also checked this for lambdas and it works as expected. But I don't think we need to add tests for that.
There was a problem hiding this comment.
what do you mean by lambdas in this case? |>? see the comment in the R/ script -- native pipe is excluded because using return() there is a parser error
There was a problem hiding this comment.
No, I meant with purrr's mappers :)
Part of #884
No hits on {lintr}.