search

RacimoLab / demes-r

R library for parsing Demes demographic models
Other
0 stars 2 forks source link

Reorganization of the R package structure and unit tests #8

Closed bodkan closed 1 year ago

bodkan commented 1 year ago

This PR implements changes to make the R package structure and organization a bit more in line with standard R package style. It also reorganizes the unit testing structure to use a cloned copy of the demes-spec in a temporary directory to avoid potential licensing issues from the CRAN side while being able to run the full suite of validation tests on the CRAN servers.

grahamgower commented 1 year ago

Thanks @bodkan! It looks like you know what you're doing here, so feel free to merge when you're ready. :)

bodkan commented 1 year ago

Thanks @grahamgower! I just had a meeting with @IsabelMarleen, discussing my reasoning for all changes and chatting about moving forward. We have also merged her latest changes from her private branch to keep everything in sync.

Things should be ready to have a full suite of tests against demes-spec validation inputs running soon. I can then help getting all tests run on the GitHub Actions pipeline in the same way that the automated CRAN checks are being done on CRAN servers.