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 2022/05/03 12:07:41 UTC

[arrow] branch master updated: ARROW-16434: [R][CI] Revert devdocs to setup-r@v1 for now

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 3c3e68c194 ARROW-16434: [R][CI] Revert devdocs to setup-r@v1 for now
3c3e68c194 is described below

commit 3c3e68c194ca6ac07086ddc1bb44fe153970213e
Author: Will Jones <wi...@gmail.com>
AuthorDate: Tue May 3 12:07:31 2022 +0000

    ARROW-16434: [R][CI] Revert devdocs to setup-r@v1 for now
    
    I'm not 100% sure of the specific cause, but also don't think it's worth getting to the exact root since we will be replacing with UCRT version soon. It seems switching to `setup-r@v2` broke our setup. My best guess is we unexpectedly were dependent on this line:
    
    https://github.com/r-lib/actions/blob/2acb5b24ed4d2f8a065b600c903d5ee62bbbe893/setup-r/src/installer.ts#L382
    
    But that we removed in v2 in:
    
    https://github.com/r-lib/actions/commit/18ab426b87dc230e2eb2d3efda516f30399f70b2
    
    Closes #13050 from wjones127/ARROW-16434-devdocs
    
    Authored-by: Will Jones <wi...@gmail.com>
    Signed-off-by: Nic Crane <th...@gmail.com>
---
 dev/tasks/r/github.devdocs.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dev/tasks/r/github.devdocs.yml b/dev/tasks/r/github.devdocs.yml
index 089953f88a..7126b14182 100644
--- a/dev/tasks/r/github.devdocs.yml
+++ b/dev/tasks/r/github.devdocs.yml
@@ -30,8 +30,8 @@ jobs:
 
     steps:
       {{ macros.github_checkout_arrow()|indent }}
-
-      - uses: r-lib/actions/setup-r@v2
+      # TODO (ARROW-16376): Switch to v2 once we use UCRT
+      - uses: r-lib/actions/setup-r@v1
         with:
           # temp workaround the fact that R 4.2 requires UCRT which isn't
           # currently part of this build.