[ci] remove unnecessary workarounds in R-package macOS jobs #6331
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In #6266, I'm working on making this project's shell scripts stricter, so they exit with a non-0 exit code the first time anything goes wrong (e.g. some variable they require isn't set or somme command they run fails).
This revealed that 2 workarounds in
.ci/test_r_package.sh
that are no longer necessary:R
to/usr/local/bin/R
(added in [ci] upgrade to R 4.1.2 in CI, change approach to macOS-latest R CI builds (fixes #4988) #4989)libomp.dylib
and Homebrew'slibomp.dylib
don't conflict with each other (added in [R-package][ci] added CI stage for R package (fixes #2335, fixes #2569) #2530)Those have both been unnecessary and not working for a while, as can be seen by messages like this in CI:
(link to recent build from master)
This removes those workarounds, simplifying the testing setup a bit.