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

Add augur version checking #6

Closed
jameshadfield opened this issue Jul 2, 2018 · 1 comment
Closed

Add augur version checking #6

jameshadfield opened this issue Jul 2, 2018 · 1 comment

Comments

@jameshadfield
Copy link
Member

The snakemake file should store the version of augur it was tested against and compare this to the currently installed augur / containerized augur version via augur --version (see nextstrain/augur#146). The code for doing this should be somewhere central (see #2).

If the augur version is out-of-date then an error should be thrown and the snakemake file exits.

If the augur version is newer, then I think a warning is displayed but the build can proceed. If we exit on this mismatch, then all the pathogen repos have to be updated every time augur updates.

@tsibley
Copy link
Member

tsibley commented Jul 2, 2018

#3 exists to consider this issue more generally (not just augur). This issue is a proposal for one approach to handling augur specifically.

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

2 participants