-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Sortmerna conda recipe #25438
base: main
Are you sure you want to change the base?
Sortmerna conda recipe #25438
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/sortmerna:
For recipes/sortmerna:
Documentation on acceptable licenses can be found here. |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipes/sortmerna:
|
I don't think we should maintain the bioconda recipe in parallel. If this one is ready here we should remove the bioconda recipe. |
I agree. Mentioned the parallel just for suggestions. How do we proceed with removing the bioconda recipe? Usual pull request? |
I was not aware of this PR. |
Created bioconda-recipes PR 48607 to remove sortmerna recipe due relocation to conda-forge |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipes/sortmerna:
|
filed issue 5391 on conda-build |
The current recipe will be maintained parallel to the Sortmerna bioconda-recipe. The bioconda recipe currently uses pre-built binaries. The current recipe performs the whole build including building 3rd party libraries. This recipe is also going to support Windows builds (not supported on bioconda).