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

Update R Tester to allow renv.lock file upload #581

Merged
merged 10 commits into from
Jan 16, 2025

Conversation

soheegoo
Copy link
Contributor

@soheegoo soheegoo commented Jan 14, 2025

Description:

  • Users can provide package dependencies for the R autotester environment by uploading a renv.lock file in addition to manually listing dependencies (existing functionality)

Changes:

  • Updated json schema to include checkbox where users specify if renv.lock file has been uploaded
  • When creating the testing environment, an R script will install the packages in the renv.lock file using renv library if the user chooses

Testing procedure:

  • Rebuild schema
  • Upload dependencies through renv.lock file and click checkbox, manually list package requirements, or both
  • Save the R Automated Test setttings
  • Run a submission for R assignment to see if successful

Tests Conducted:

  • Uploading one dependency through all use cases: knitr package was 1) entered manually as a package requirement, 2) specified through a renv.lock file, and 3) uploaded through both methods at the same time
  • Uploading multiple dependencies through both methods combined: renv.lock file (knitr package) and manual input (plotly package)

@soheegoo soheegoo marked this pull request as draft January 14, 2025 19:54
@soheegoo soheegoo marked this pull request as ready for review January 14, 2025 19:55
@donny-wong donny-wong self-requested a review January 16, 2025 15:47
@soheegoo soheegoo requested a review from david-yz-liu January 16, 2025 20:31
@david-yz-liu david-yz-liu merged commit de4c719 into MarkUsProject:master Jan 16, 2025
7 checks passed
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.

3 participants