phac-nml / biohansel

Rapidly subtype microbial genomes using single-nucleotide variant (SNV) subtyping schemes
Apache License 2.0
25 stars 7 forks source link

Release v2.4.0 #121

Closed peterk87 closed 4 years ago

peterk87 commented 4 years ago

Change log for v2.4.0 release:

DarianHole commented 4 years ago

Hi @peterk87,

Everything seems to be in order on the code side. Pytests run on both 3.6 and 3.7 as expected.

Although I think I've found an installation issue I missed last time (or I don't know how to install). Currently I pip install from the branch (pip install .) and when I do that, the aho_corasick and qc directories are not being copied with the data (no issue when I tried 2.3 or 2.2 in this).

I'll see if its on my end a bit more as well

peterk87 commented 4 years ago

Thanks a lot @DarianHole for taking a look at this PR! I fixed the packaging issue and made the suggested changes.

I think it should auto-deploy when a release tag is created.

https://github.com/phac-nml/biohansel/blob/f653c2f6aef0c8a93b2553de9bf926940894bafe/.travis.yml#L13-L23

The user and secure password could be changed to the phac-nml and a secure password generated by the travis-ci cli tool. I've added the phac-nml user as a project owner on PyPI.

DarianHole commented 4 years ago

@peterk87 I am not authorized to merge this one but its good. Let me know if anything else is needed!

peterk87 commented 4 years ago

Thanks a lot @DarianHole! I'll create a v2.4.0 release and make sure it's up on PyPI

glabbe commented 4 years ago

Thanks a lot @peterk87 and @DarianHole for this new version release, much appreciated!