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 Autometa #21480

Closed
wants to merge 8 commits into from
Closed

Add Autometa #21480

wants to merge 8 commits into from

Conversation

evanroyrees
Copy link
Contributor

Addition of Autometa 2.0.0-alpha version


Please use the following BiocondaBot commands:

Everyone has access to the following BiocondaBot commands, which can be given in a comment:

@BiocondaBot please update Merge the master branch into a PR.
@BiocondaBot please add label Add the please review & merge label.
@BiocondaBot please fetch artifacts Post links to CI-built packages/containers.
You can use this to test packages locally.

For members of the Bioconda project, the following command is also available:

@BiocondaBot please merge Upload built packages/containers and merge a PR.
Someone must approve a PR first!
This reduces CI build time by reusing built artifacts.

Also, the bot watches for comments from non-members that include @bioconda/<team> and will automatically re-post them to notify the addressed <team>.

@evanroyrees
Copy link
Contributor Author

@BiocondaBot please add label

@BiocondaBot BiocondaBot added the please review & merge set to ask for merge label Apr 13, 2020
Copy link
Contributor

@dpryan79 dpryan79 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see comments.

- diamond
- hmmer
- hdbscan
- openssl ==1.0.*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a compelling reason for why you need to pin to this particular version rather than using the version we pin to in every other package?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is pinned for samtools to run properly

@@ -0,0 +1,68 @@
{% set name = "Autometa" %}
{% set sha256 = "b9b82c15e6f4d76c49d3e80d38a901606e7fd8ba945f83657535d27bb3d8860b" %}
{% set version = "2.0.0a0" %}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We try not to package alpha releases. If you want this in bioconda please tag an proper release version.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is my first time contributing to bioconda, so I have been going through this as a sort of trial to see if I can get this to work. When we release a production ready version, I will update. Should the PR be closed until then? We currently do not have a production ready version available.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, we might as well close it until then. Definitely keep your fork though, since it looks like a minor update should get this ready for distribution once your release is tagged :)

recipes/autometa/meta.yaml Show resolved Hide resolved
recipes/autometa/setup.py Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
please review & merge set to ask for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants