raulcd commented on PR #45524:
URL: https://github.com/apache/arrow/pull/45524#issuecomment-2656635294
Once we disable `return_linter` there are other linters that fail:
```
tests/testthat/test-dplyr-collapse.R:146:7: style: [commented_code_linter]
Remove commented code.
# filter(dbl > 2) %>%
^~~~~~~~~~~~~~~~~~~
tests/testthat/test-dplyr-collapse.R:147:7: style: [commented_code_linter]
Remove commented code.
# select(chr, int, lgl) %>%
^~~~~~~~~~~~~~~~~~~~~~~~~
tests/testthat/test-dplyr-collapse.R:148:7: style: [commented_code_linter]
Remove commented code.
# mutate(twice = int * 2L) %>%
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
tests/testthat/test-dplyr-collapse.R:149:7: style: [commented_code_linter]
Remove commented code.
# group_by(lgl) %>%
^~~~~~~~~~~~~~~~~
tests/testthat/test-dplyr-collapse.R:150:7: style: [commented_code_linter]
Remove commented code.
# summarize(total = sum(int, na.rm = TRUE)) %>%
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In addition: Warning message:
```
I think the easier approach here is to temporarily pin `lintr` to the
version we were using and open a new issue to unpin it and fix the new linters
so we can fix CI which is broken at the moment.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
