[arrow] branch master updated: ARROW-16011: [R] CI jobs should fail if lintr picked up issues

[th...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new cf53e3cdcb ARROW-16011: [R] CI jobs should fail if lintr picked up issues
cf53e3cdcb is described below

commit cf53e3cdcb4b2d68bc95f85b9ebcc0a212eb6ed1
Author: assignUser <ja...@wujciak.de>
AuthorDate: Wed Apr 6 11:41:32 2022 +0100

    ARROW-16011: [R] CI jobs should fail if lintr picked up issues
    
    Building on @thisisnic 's work  #12696
    This requires less maintenance in the workflows but does add a `Suggests` on lintr
    
    Closes #12790 from assignUser/ARROW-16011
    
    Lead-authored-by: assignUser <ja...@wujciak.de>
    Co-authored-by: Jacob Wujciak-Jens <ja...@wujciak.de>
    Co-authored-by: Nicola Crane <th...@gmail.com>
    Signed-off-by: Nic Crane <th...@gmail.com>
---
 .github/workflows/r.yml | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/.github/workflows/r.yml b/.github/workflows/r.yml
index 81fdcb63ec..1f27d4fa3b 100644
--- a/.github/workflows/r.yml
+++ b/.github/workflows/r.yml
@@ -382,6 +382,15 @@ jobs:
             check_dir = 'check',
             timeout = 3600
           )
+      - name: Run lintr
+        env: 
+          NOT_CRAN: "true"
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        shell: Rscript {0}
+        working-directory: r
+        run: |
+          remotes::install_github("jonkeane/lintr@arrow-branch")
+          lintr::expect_lint_free()
       - name: Dump install logs
         shell: cmd
         run: cat r/check/arrow.Rcheck/00install.out