jonkeane commented on PR #13620: URL: https://github.com/apache/arrow/pull/13620#issuecomment-1202467777
Interesting, thanks for that additional context. We should merge this fix regardless (since it's the recommended way of running lintr!), but I suspect that something else might have happened with the seemingly off files here. I was curios, so went looking and turns out, the offending whitespace line was introduced in https://github.com/apache/arrow/pull/13610 which did get properly flagged: https://github.com/apache/arrow/runs/7349390401?check_suite_focus=true so at least that did work at the time. Anyway, like I said, I'll merge this since it's the right way to be running lintr according to the developers -- 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]
