{bio}[foss/2020a] MetaPhlAn v3.0.9 w/ Python 3.8.2#12917
{bio}[foss/2020a] MetaPhlAn v3.0.9 w/ Python 3.8.2#12917lexming merged 14 commits intoeasybuilders:developfrom
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@branfosj can we rebuild this one. Thx |
Can you run |
This comment has been minimized.
This comment has been minimized.
…asyconfigs into 20210521143242_new_pr_MetaPhlAn309
No worries, done. Thx |
|
@robqiao could you address the test failures and see if it can use the slightly order version of |
|
Test report by @boegel |
|
@robqiao |
|
deps on #14326 |
…asyconfigs into 20210521143242_new_pr_MetaPhlAn309
This comment has been minimized.
This comment has been minimized.
…asyconfigs into 20210521143242_new_pr_MetaPhlAn309
|
deps on PR |
|
|
Test report by @robqiao |
This comment has been minimized.
This comment has been minimized.
…asyconfigs into 20210521143242_new_pr_MetaPhlAn309
|
@boegelbot: please test @ generoso |
|
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 997086048 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @lexming |
|
@boegelbot: please test @ generoso |
|
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 997090778 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
lexming
left a comment
There was a problem hiding this comment.
Thanks for the update @manifestoso . I just made a little fix to the sanity checks, as long as you already check for files inside bin and lib ther is no need to explicitly check for those folders. In this case, you already check for the executables inside bin and since MetaPhlAn is a Python package we must have a site-packages inside lib.
LGTM
|
Going in, thanks @manifestoso ! |
(created using
eb --new-pr)