You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by th...@apache.org on 2023/07/20 17:03:19 UTC

[arrow] branch main updated: GH-36787: [R] lintr update leads to failing tests on main (#36788)

This is an automated email from the ASF dual-hosted git repository.

thisisnic pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new fca6a66333 GH-36787: [R] lintr update leads to failing tests on main (#36788)
fca6a66333 is described below

commit fca6a6633300694ae9089e92b736ce7d95fd7ed7
Author: Nic Crane <th...@gmail.com>
AuthorDate: Thu Jul 20 18:03:12 2023 +0100

    GH-36787: [R] lintr update leads to failing tests on main (#36788)
    
    ### What changes are included in this PR?
    
    Turning off the newly introduced indentation linter as it causes test failures, and isn't in sync with styler which we use to style our code.
    
    ### Are these changes tested?
    
    No, is linter config.
    
    ### Are there any user-facing changes?
    
    No
    * Closes: #36787
    
    Authored-by: Nic Crane <th...@gmail.com>
    Signed-off-by: Nic Crane <th...@gmail.com>
---
 r/.lintr | 1 +
 1 file changed, 1 insertion(+)

diff --git a/r/.lintr b/r/.lintr
index 1bd80aff4c..085ff45123 100644
--- a/r/.lintr
+++ b/r/.lintr
@@ -15,6 +15,7 @@ license:  #  Licensed to the Apache Software Foundation (ASF) under one
   #  specific language governing permissions and limitations
   #  under the License.
 linters: linters_with_defaults(
+  indentation_linter = NULL,
   line_length_linter = line_length_linter(120),
   object_name_linter = NULL,
   # Even with a liberal definition of name styles, some of our names cause issues due to `.`s for s3 classes or NA in the name