https://nf-co.re logo
Join Slack
Powered by
Yes, it always runs the dev version
# tools
m
Yes, it always runs the dev version
👍 1
n
I'm having some issues with the 
--fix
option in this PR: github.com/nf-core/modules/pull/12163. It creates invalid YAML keys like this:
Copy code
modules/nf-core/expansionhunter/meta.yml: SyntaxError: Implicit map keys need to be on a single line (110:7)
              108 |         type: string
              109 |         description: The tool name
            > 110 |     - ExpansionHunter --version | head -1 | sed -n 's/^.*ExpansionHunter v//;
                  |       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
            > 111 |         s/]//p':
                  | ^^^^^^^^^^^^^^^^
              112 |         type: eval
              113 |         description: The expression to obtain the version of the tool
              114 |   versions_bgzip:
quick look 1
Looks like we also need to update the devcontainer in the modules repo:
Copy code
WARNING  TOWER_ACCESS_TOKEN is not set. Wave API requests will be subject to stricter rate limits.
         To increase your quota, set the TOWER_ACCESS_TOKEN environment variable with your Seqera Platform token.
         See <https://docs.seqera.io/wave/api#api-limits> for more information.
WARNING  Failed to build docker container for linux/arm64: It looks like wave is not installed. Please ensure it is available in your PATH..
         This is only critical if the tool should support arm64.
ERROR    Failed to build singularity container for linux/amd64: It looks like wave is not installed. Please ensure it is available in your
         PATH.
ERROR    Failed to build docker container for linux/amd64: It looks like wave is not installed. Please ensure it is available in your PATH.
WARNING  Failed to build singularity container for linux/arm64: It looks like wave is not installed. Please ensure it is available in your
         PATH.. This is only critical if the tool should support arm64.
ERROR    No docker image found for linux/amd64

ERROR    ✗ Some container builds failed for expansionhunter
e
replying here just to get pinged with the fix answer xD
oh it was my thread 😛
n
@mashehu fine if I just force-merge now? I need this update and the containers implementation isn't used in pipelines yet
👍🏻 1
m
will enable the bypass for a bit then 😬
n
Thanks 😉
m
the doors are open
n
shared a GIF
(thank you!)
2 Views
Open in Slack
PreviousNext
Powered by