Skip to content

Migrate from Travis CI to GitHub Actions#78

Open
nayeonshin wants to merge 21 commits intoEnd-to-end-provenance:masterfrom
nayeonshin:nshin_migrate-to-ga
Open

Migrate from Travis CI to GitHub Actions#78
nayeonshin wants to merge 21 commits intoEnd-to-end-provenance:masterfrom
nayeonshin:nshin_migrate-to-ga

Conversation

@nayeonshin
Copy link
Copy Markdown
Collaborator

@nayeonshin nayeonshin commented Apr 21, 2022

Note: My VSCode automatically formatted README.md.

@nayeonshin nayeonshin changed the title ci: migrate from Travis CI to GitHub Actions Migrate from Travis CI to GitHub Actions Apr 21, 2022
@nayeonshin nayeonshin added the enhancement New feature or request label Apr 21, 2022
@nayeonshin nayeonshin marked this pull request as draft April 21, 2022 19:49
@nayeonshin nayeonshin marked this pull request as ready for review April 21, 2022 20:02
@nayeonshin nayeonshin force-pushed the nshin_migrate-to-ga branch from 0fd57b7 to d72b349 Compare April 28, 2022 19:50
@nayeonshin nayeonshin force-pushed the nshin_migrate-to-ga branch from dcb005d to e8015fb Compare April 28, 2022 20:45
@nayeonshin
Copy link
Copy Markdown
Collaborator Author

@blernermhc Hi! Have you heard back from anyone on the data provenance team (because I think you emailed someone regarding this issue)? I tried a bunch of different things to R-CMD-check.yaml, but somehow none of them worked.

@blernermhc
Copy link
Copy Markdown
Member

Hi Nayeon,

Sorry for being so slow in responding. It has taken a while to wind down the semester and get back to doing research. I don't think I ever actually sent any email. I would love to take what you have done and continue working on it, but I do not want to merge this into the master branch. It would be better to be able to push this to a new branch. Are you able to do that?

Barbara

@blernermhc
Copy link
Copy Markdown
Member

Hi Nayeon,

I figured out how to do this, at least it worked for the provTraceR package, which I believe faced the same issues. Try this:

In the setup-r section, add:
extra-repositories: https://end-to-end-provenance.github.io/drat/

Remove from the r-setup-dependencies section:
package=rdt?ignore, package=rdtLite?ignore

Remove from the check-r-packages section:
R_CHECK_FORCE_SUGGESTS: false

I have the following in the check-r-packages section. It was created with the usethis command:
upload-snapshots: true

I would suggest trying it with those changes. I also ran into a problem that it could not build the XML package. I don't know if that will be an issue for provDebugR, but if it is, change the extra-packages line to this:
extra-packages: any::rcmdcheck, any::XML

@nayeonshin
Copy link
Copy Markdown
Collaborator Author

nayeonshin commented Jun 2, 2022

@blernermhc Hi! No need to be sorry. I hope you've been resting well. I actually didn't expect that you'd respond during the summer. Yes I can definitely close this PR and push the changes into a new branch.

Before opening a new branch, I'll first try doing what you suggested and let you know how it goes.

Thank you so much!
Nayeon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants