jonkeane commented on a change in pull request #12395:
URL: https://github.com/apache/arrow/pull/12395#discussion_r804003030
##########
File path: ci/scripts/r_test.sh
##########
@@ -26,6 +26,12 @@ pushd ${source_dir}
printenv
+# Before release, we always copy the relevant parts of the cpp source into the
+# package. In some CI checks, we will use this version of the source:
+# this is done by setting ARROW_SOURCE_HOME to something other than "/arrow"
+# (which is where the arrow git checkout is found in docker and other CI jobs)
Review comment:
```suggestion
# Before release, we always copy the relevant parts of the cpp source into
the
# package. In some CI checks, we will use this version of the source:
# this is done by setting ARROW_SOURCE_HOME to something other than "/arrow"
# (which is where the arrow git checkout is found in docker and other CI
jobs)
# In the other CI checks the files are synced but ignored.
```
🙃 but yeah this is a good description with what one needs to grep for if it
fails. One small addition
--
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]
