-
Notifications
You must be signed in to change notification settings - Fork 8
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
don't use container #3
base: master
Are you sure you want to change the base?
Conversation
@robkooper looks like tests are failing. Should we force merge this PR and have @tezansahu address the issues with the tests in his open PR, or do you want to try and fix the tests here? |
need to add default account and xml package for tests. |
Co-authored-by: Alexey Shiklomanov <[email protected]>
Looks like the problem is that GH Actions is running this with https://github.com/PecanProject/rpecanapi/runs/1350179239#step:7:9 Might be able to get away with something hacky like using |
# entrypoint: /home/bety/script/betyuser.sh | ||
args: user carya illinois "Carya Demo User" [email protected] 1 1 |
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.
FWIW, I'm currently testing this out on a personal branch of this -- https://github.com/ashiklom/rpecanapi/runs/1386059263?check_suite_focus=true
UPDATE: This made it past this line, but failed on the actual R CMD check
step.
# entrypoint: /home/bety/script/betyuser.sh | |
args: user carya illinois "Carya Demo User" [email protected] 1 1 | |
entrypoint: /bin/bash | |
args: /home/bety/script/betyuser.sh carya illinois "Carya Demo User" [email protected] 1 1 |
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.
Though, I guess setting entrypoint: /home/bety/script/betyuser.sh
might be a more straightforward version of the same thing.
trying to fix github actions build