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.

Sign up for GitHub

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

Jump to bottom

Workflow test #9

Merged
merged 38 commits into from
Sep 9, 2024
Merged

Workflow test #9

merged 38 commits into from
Sep 9, 2024

Conversation

mduncans
Copy link
Collaborator

@mduncans mduncans commented Sep 9, 2024

No description provided.

@mduncans
updated gh actions yaml to download suggested packages too.
8a27109
@mduncans
trying to get bbr installed for vignettes needs for gh actions.
753c973
@mduncans
test for workflows
8ad9c09
@mduncans
installing bbr before setup-r-dependencies
b7eebc1
@mduncans
added remotes
ddf1e94
@mduncans
more bbr dependencies
d907146
@mduncans
trying to add bbr to extra-packages
78149c7
@mduncans
fixed spelling
05a0cc1
@mduncans
added bbr dependencies
d71ed22
@mduncans
added error handling around processx::run("squeue"). Will throw warni…
930a989 
…ng and likely errors because option never gets set.
@mduncans
updated error handling messages. set vignettes to fales.
f32351f
@mduncans
typo fix
dc8665f
@mduncans
stopping vignettes
b75aede
@mduncans
another typo
b6e8bc7
@mduncans
different args
b4b6588
@mduncans
prebuilding vignettes to prevent gh actions from rebuilding them.
dff4ee2
@mduncans
why not just install slurm?
6efb776
@mduncans
back to allowing building of vignettes because the error still revolv…
44a6b52 
…es around slurm.
@mduncans
seeing if sinfo/sacct run after download adding env to r cmd check
4b2cff3
@mduncans
removed env for now
75bbc1c
@mduncans
sacct showed reasonable output
15c06eb
@mduncans
added sacct outputs to github_workspace
38fb9cf
@mduncans
sacct is still running in gh actions even though only sinfo was calle…
6c2edf5 
…d? I think outputs directories need to be rename?
@mduncans
sacct and sinfo were both printed due to echo to GITHUB_PATH. Trying …
c650f60 
…to call sinfo with processx
@mduncans
didn't know processx wasn't included
4c04732
@mduncans
removed outputs directory moves beacuse sinfo seemed to work without …
58dd857 
…it? if both sinfo and sacct work without the move going to add --json args
@mduncans
sacct is showing no output, but sinfo is.
c8cd335
@mduncans
args = --json works for both. moving the call to sinfo and sacct to d…
13117be 
…ifferent step
@mduncans
added sys.which("sinfo")/squeue
ffa8c34
@mduncans
changed chmod to 755
f159377
@mduncans mduncans merged commit 531ea0e into main Sep 9, 2024
2 checks passed
@wes-a2ai wes-a2ai deleted the workflow_test branch September 9, 2024 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mduncans