Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Revise R tests. #8430

Merged
merged 48 commits into from
Nov 9, 2022
Merged

[CI] Revise R tests. #8430

merged 48 commits into from
Nov 9, 2022

Conversation

trivialfis
Copy link
Member

@trivialfis trivialfis commented Nov 6, 2022

  • Use the standard package check (check on the tarball instead of the source tree).
  • Run commands in parallel.
  • Cleanup dependencies installation.
  • Replace makefile.
  • Documentation.
  • Test using the image from rhub.

Close #8429 .

- Use the standard package check (check on the tarball instead of the source tree).
- Run commands in parallel.
- Cleanup dependencies installation.
- Replace makefile.
- Documentation.
- Optional test with rhub.
.github/workflows/r_rhub.yml Outdated Show resolved Hide resolved
R-package/tests/helper_scripts/install_deps.R Outdated Show resolved Hide resolved
R-package/tests/helper_scripts/install_deps.R Outdated Show resolved Hide resolved
@trivialfis
Copy link
Member Author

trivialfis commented Nov 6, 2022

/gha run rhub-test

@trivialfis
Copy link
Member Author

trivialfis commented Nov 6, 2022

run rhub-test

doc/contrib/coding_guide.rst Outdated Show resolved Hide resolved
@trivialfis trivialfis merged commit a83748e into dmlc:master Nov 9, 2022
@trivialfis trivialfis deleted the r-checks branch November 9, 2022 01:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] Revise R tests on github action
2 participants