CFSAN-Biostatistics/shigatyper

version discrepancy

Closed this issue · 5 comments

CC @rpetit3

In the StaPH-B docker image for shigatyper v2.0.1, I'm seeing this output:

$ docker run staphb/shigatyper:2.0.1 shigatyper --version
ShigaTyper 2.0.0

instead of the expected ShigaTyper 2.0.1

The docker image is built with this dockerfile, which pulls the tag conda-package-2.0.1 from GitHub and builds/installs shigatyper via running python3 setup.py install

When looking at the tag for conda-package-2.0.1, the shigatyper.py code shows 2.0.0:

version = "2.0.0"

Could the code for the command shigatyper --version be updated to use the version from setup.py instead of what is listed in shigatyper.py?

I would give it a try myself and submit a PR but I'm afraid I would not be following the best/proper python practices for versioning & muddy the waters further

#9 (comment)

Easiest would be fix and rerelease, but changing that line should do it

I am all for doing whatever is easiest 😄

Ok, taking the opportunity (while fixing #13 ) to do version numbering correctly - shigatyper --version should return the correct patch version everywhere starting with release 2.0.2 (to come this week.)

Fixed in 2.0.2.

Thanks, can confirm it prints ShigaTyper 2.0.2 as expected now. Thanks!