Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/19290#discussion_r140244036 --- Diff: dev/lint-r.R --- @@ -28,6 +28,7 @@ if (! library(SparkR, lib.loc = LOCAL_LIB_LOC, logical.return = TRUE)) { # NOTE: The CRAN's version is too old to adapt to our rules. if ("lintr" %in% row.names(installed.packages()) == FALSE) { devtools::install_github("jimhester/lintr@a769c0b") + # devtools::install_github("jimhester/lintr@5431140") --- End diff -- Oh, just to be clear, I didn't change this yet but just commented this for now ... Actually, `devtools::install_github("jimhester/lintr@5431140")` is new one but I didn't change this yet just in case. Will add some comments around this when I add few more changes and clean up soon in any event.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org