dongjoon-hyun commented on a change in pull request #26564: [SPARK-29936][R]
Fix SparkR lint errors and add lint-r GitHub Action
URL: https://github.com/apache/spark/pull/26564#discussion_r347198034
##########
File path: .github/workflows/master.yml
##########
@@ -72,3 +72,26 @@ jobs:
run: ./dev/check-license
- name: Dependencies
run: ./dev/test-dependencies.sh
+
+ lintr:
+ runs-on: ubuntu-latest
+ name: Linter (R)
+ steps:
+ - uses: actions/checkout@master
+ - uses: actions/setup-java@v1
+ with:
+ java-version: '11'
+ - name: install R
+ run: |
+ echo 'deb https://cloud.r-project.org/bin/linux/ubuntu bionic-cran35/'
| sudo tee -a /etc/apt/sources.list
+ sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys
E298A3A825C0D65DFD57CBB651716619E084DAB9
+ sudo apt-get update
+ sudo apt-get install -y r-base r-base-dev libcurl4-openssl-dev
+ - name: install R packages
+ run: |
+ sudo Rscript -e "install.packages(c('curl', 'xml2', 'httr',
'devtools', 'testthat', 'knitr', 'rmarkdown', 'roxygen2', 'e1071', 'survival'),
repos='https://cloud.r-project.org/')"
+ sudo Rscript -e "devtools::install_github('jimhester/lintr')"
Review comment:
I didn't remove that because we still have Jenkins and other environments
which we cannot install R and packages. We only can handle GitHubAction and
AppVeyor.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]