-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Replace t_coffee with t-coffee #38199
Conversation
Can you please fix the linting issues (bump the build number and remove the |
I think this PR should also be marked as resolving the issue #32576. I'm excited to see it land! |
OK, did I properly address what you meant? |
@amizeranschi Yes, thanks, let's see if tests pass now! |
@dpryan79, are we sure that the new
|
I see similar errors with the Galaxy wrapper, it seems a recent rebuild is causing issues (it was definitely working before with the same t-coffee version). |
Yes, the t-coffee package build was not complete and that's what doomed this package's test. I described the bug and made a pull request to fully build t-coffee. |
Can we re-run the builds now that the new |
In light of the recent un-blacklisting and update of `t-coffee` (a78d91b), it should now replace `t_coffee` as a dependency here. For more info: #32576 (comment)
@BiocondaBot please merge |
I will attempt to upload artifacts and merge this PR. This may take some time, please have patience. |
Merge PR bioconda#38199, commits were: * Remove commented out line * Update meta.yaml * Update meta.yaml * Replace t_coffee with t-coffee In light of the recent un-blacklisting and update of `t-coffee` (bioconda@a78d91b), it should now replace `t_coffee` as a dependency here. For more info: bioconda#32576 (comment)
In light of the recent un-blacklisting and update of
t-coffee
(a78d91b), it should now replacet_coffee
as a dependency here.For more info: #32576 (comment)