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

Handle megares 2.0.0 #295

Merged
merged 6 commits into from
May 14, 2020

Conversation

martinghunt
Copy link
Collaborator

Fixes issues #287 and #294.

megares changed the filenames and format of their download between versions 1.0.1 and 2.0.0. This PR fixes ariba getref so that it works for all available versions (<=2.0.0) of megares at the time of writing.

@puethe puethe mentioned this pull request May 14, 2020
@puethe
Copy link
Contributor

puethe commented May 14, 2020

Thanks Martin, this is great! Are you taking over support again? :-P
Any chance to get the build passing before merging? Looks like there's just a Python dependency missing, probably something with versions.
Cheers,
Christoph

@martinghunt
Copy link
Collaborator Author

@puethe I don't have time to support properly, sorry. Can make time for important bug fixes like this one though! Will get travis working - hopefully won't take too many commits...

@martinghunt
Copy link
Collaborator Author

There you go. If you're happy to merge, can you make a new release please? I've no idea what the release process is now.

@puethe
Copy link
Contributor

puethe commented May 14, 2020

Thanks a lot @martinghunt . Will merge and increment the patch number. Release process - DockerHub and BioConda should pick up automatically. PyPi is manual, but can do that.

@puethe puethe merged commit abf9c66 into sanger-pathogens:master May 14, 2020
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

Successfully merging this pull request may close these issues.

2 participants