Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19959#discussion_r156529170
--- Diff: dev/lint-r.R ---
@@ -27,10 +27,11 @@ if (! library(SparkR, lib.loc = LOCAL_LIB_LOC,
logical.return = TRUE)) {
# Installs lintr from Github in a local directory.
# NOTE: The CRAN's version is too old to adapt to our rules.
if ("lintr" %in% row.names(installed.packages()) == FALSE) {
--- End diff --
Yea, that sounds ok to me; however, I remember @shivaram had a different
idea(?) (did I remember this correctly .. ?)
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]