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.

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

Downstream dependency testing #6

Open
fozy81 opened this issue Sep 5, 2017 · 3 comments
Open

Downstream dependency testing #6

fozy81 opened this issue Sep 5, 2017 · 3 comments

Comments

@fozy81
Copy link
Collaborator

fozy81 commented Sep 5, 2017

This is perhaps an issue for ALA4R repo to add downstream tests, so that changes to ALA4R will not break NBN4R. And if they do break because of changes to API, downstream packages can be notified that updates need to be made. I believe there are other packages like NBN4R for particular countries using the ALA infrastructure.

@AugustT
Copy link
Member

AugustT commented Sep 6, 2017

Good call

@AugustT
Copy link
Member

AugustT commented Sep 6, 2017

one for @raymondben to think about

@raymondben
Copy link

ALA4R probably doesn't need to explicitly add tests of downstream packages, we just need to make sure reverse dependencies still pass their own tests (as CRAN policies require already). https://github.com/yihui/crandalf may be a good way to automate this. But as NBN4R development progresses, I am sure that we'll find aspects of ALA4R that are particularly important for NBN4R but which aren't yet tested in the ALA4R tests, and which we'll thus want to add. So let's leave this issue open as a reminder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants