Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/19290
  
    For more context, I believe `lintr` was initially installed in 
https://github.com/apache/spark/commit/004f57374b98c4df32d9f1e19221f68e92639a49.
 Upgrade to jimhester/lintr@a769c0b was proposed here in 
https://github.com/apache/spark/commit/7d1175011c976756efcd4e4e4f70a8fd6f287026 
but I believe this didn't actually upgrade the package on Jenkins as `lintr` 
was already installed  and `"lintr" %in% row.names(installed.packages())  == 
FALSE` gives `FALSE`.
    
    So, I believe `lintr` in Jenkins is already not synced to 
jimhester/lintr@a769c0b as fixed in 
https://github.com/apache/spark/commit/7d1175011c976756efcd4e4e4f70a8fd6f287026 
unless it was manually upgraded afterwards (I can't check but this is just my 
assumption).
    
    I believe eventually we should upgrade and match with this anyway to detect 
project's R style in each PR. I at least have been in a PR where I had to 
manually suggest to fix.
    
    



---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to