raulcd commented on code in PR #13453:
URL: https://github.com/apache/arrow/pull/13453#discussion_r911777504
##########
.github/workflows/r_nightly.yml:
##########
@@ -101,18 +127,37 @@ jobs:
# strip superfluous nested dirs
new_paths <- sub(art_path, ".", new_paths)
dirs <- dirname(new_paths)
- dir_result <- sapply(dirs, dir.create, recursive = TRUE)
-
- if (!all(dir_result)) {
- stop("There was an issue while creating the folders!")
- }
+ sapply(dirs, dir.create, recursive = TRUE, showWarnings = FALSE)
- copy_result <- file.copy(current_path, new_paths)
+ # overwrite allows us to "force push" a new version with the same
name
+ copy_result <- file.copy(current_path, new_paths, overwrite = TRUE)
if (!all(copy_result)) {
stop("There was an issue while copying the files!")
}
+ - name: Prune Repository
+ shell: bash
+ env:
+ KEEP: ${{ github.event.inputs.keep || 14 }}
Review Comment:
shouldn't this already be the default value? I am not sure I understand why
do we need the `|| 14`
--
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]
