bedatadriven / activityinfo-R

ActivityInfo R Language Client
https://www.activityinfo.org/support/docs/R/
17 stars 12 forks source link

Rationalize GitHub checks #116

Closed akbertram closed 3 months ago

akbertram commented 6 months ago

We have one passing check, and the others are systematically failing. Can we consolidate? Do we need cross platform testing? At least get everything passing reliably.

nickdickinson commented 3 months ago

Everything is defined in the GHA workflow R-CMD-check. Tests are run using windows-latest and ubuntu-latest. I tested separately on MacOS and could not find why the GHA didn't work on MacOS so leaving out. If we find it is required, I can test further for other GHA that could be used for MacOS. I think we'll cover most edge cases with two OSs.

Note, it is also possible to manually trigger a test in a rocker container, which skips the longer R-CMD-check. This is the GHA called activityinfo-R-package-tests