You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@arrow.apache.org by "Jonathan Keane (Jira)" <ji...@apache.org> on 2021/08/03 14:29:00 UTC

[jira] [Created] (ARROW-13537) [R] Enable object name linter

Jonathan Keane created ARROW-13537:
--------------------------------------

             Summary: [R] Enable object name linter
                 Key: ARROW-13537
                 URL: https://issues.apache.org/jira/browse/ARROW-13537
             Project: Apache Arrow
          Issue Type: Bug
          Components: R
            Reporter: Jonathan Keane


We don't currently even try to use the object name lintr because identifying the S3 methods isn't quite right for Arrow in lintr. The basics of this is that we should be able to do something like `head.ArrowClassName <- function()` but if the `head` bit isn't correctly identified as an S3 method, lintr will complain thinking we are mixing dots and camel case. There are a number of different ways this doesn't fully work (see below). Once those are resolved, we should be able [to enable an object name linter as well|https://github.com/apache/arrow/blob/a52050a8617a3d1164ea3e960b0a0260023d98bd/r/.lintr#L22]

A few PRs + issues around this:
https://github.com/jimhester/lintr/pull/843
https://github.com/jimhester/lintr/pull/841
https://github.com/jimhester/lintr/issues/842
https://github.com/jimhester/lintr/issues/846



--
This message was sent by Atlassian Jira
(v8.3.4#803005)