thisisnic commented on a change in pull request #10261:
URL: https://github.com/apache/arrow/pull/10261#discussion_r627992188
##########
File path: r/tests/testthat/test-RecordBatch.R
##########
@@ -15,7 +15,7 @@
# specific language governing permissions and limitations
# under the License.
-context("RecordBatch")
+library(dplyr)
Review comment:
In PRs on other tests, I think some people have been removing `context`
as it doesn't add anything useful to the CI output, so I've just been getting
rid of it when I've been working with certain files. Happy to add back in, but
sounds like more discussion needed?
Cool, won't import `dplyr` here.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
