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
Use R2U to install r packages [no version bump] #495
base: main
Are you sure you want to change the base?
Conversation
6afa10b
to
f8d1170
Compare
with the github.event.pull_request.head.sha we fetch the real last commit during a PR, rather than the temporary commit. - uses: actions/checkout@v3
with:
# pull_request ref https://github.com/orgs/community/discussions/28474
ref: ${{ github.event.pull_request.head.sha }}
Hence, we are removing the check on PR
|
Can you correct the title, authors and description to match: https://www.biorxiv.org/content/10.1101/332783v3.full |
Setup r2u using https://eddelbuettel.github.io/r-ci/run.sh sets ENV USE_BSPM=${USE_BSPM:-"TRUE"} and remove RSPM actions/checkout@v3 should checkout real branch commits even on PR ref: ${{ github.event.pull_request.head.sha }} --no-pager, quit after getting log info Use DESCRIPTION to Install R dependencies. Update version in DESCRIPTION using desc::desc_set_version Co-authored-by: Glenda <gmyenni@users.noreply.github.com> fixes #482
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the tests aren't running? Not sure if that's because there are so many notes and warnings in the R CMD check before it gets to run the actual tests.
Where is the update_data step?
3396b1d
to
cae33a4
Compare
I had missed update_data step. I have added it back and I made sure the tests run. |
|
Also, the description in the the Description still needs to be updated. You can just use the abstract from the paper. |
No description provided.