ARROW-15690: [Dev] Update GitHub Actions workflows that hardcode master as default #12436
Conversation
|
I'll back out the R windows job changes for now, they're unrelated (they're about the r-lib/actions tag, not our master branch). It looks like pacman isn't on the path in the v2 workflows (r-lib/actions@master...v2-branch#diff-3e2a1a7f497c1884dc161d6e07fdc58e4853eb91bf13034122e3814b501df0d5L382), I'll follow up with that upstream. |
ci/scripts/install_kartothek.sh
Outdated
There was a problem hiding this comment.
Did you check that no existing CI job was passing "master" to this script?
There was a problem hiding this comment.
I see, it's simply redundant with the else branch below?
There was a problem hiding this comment.
Yes, we can safely remove this branch.
ci/scripts/install_turbodbc.sh
Outdated
|
@nealrichardson it looks good to me, but the R builds are failing. |
nealrichardson
force-pushed
the
default-branch-1
branch
from
755d08d to
04e7b3b
Compare
@kszucs I fixed the issue upstream, builds passed this time |
|
Benchmark runs are scheduled for baseline = e4e866f and contender = ee9354d. ee9354d is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
4 participants
Included a few other master-related cleanups I saw