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

Update Singularity build command in installation.md #2273

Merged
merged 4 commits into from Jan 23, 2024
Merged

Conversation

kmlhpk
Copy link
Contributor

@kmlhpk kmlhpk commented Jan 22, 2024

Just a small change to the Singularity section of the install instructions:

docker://multiqc/multiqc:1.20 is not the right URI, need to use v1.20 (replacing version as appropriate) as the tag instead

Using the former command results in FATAL: While performing build: conveyor failed to get: reading manifest 1.20 in docker.io/multiqc/multiqc: manifest unknown: manifest unknown

Using v1.19 instead worked.

  • This comment contains a description of changes (with reason)

Copy link
Member

@ewels ewels left a comment

Choose a reason for hiding this comment

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

Great spot, thank you! I noticed another instance of the same typo further up too, for Docker. So fixed that and pushed to your PR..

@ewels ewels merged commit fce70fb into MultiQC:main Jan 23, 2024
3 checks passed
@kmlhpk
Copy link
Contributor Author

kmlhpk commented Jan 25, 2024

cool, thanks! happy to have helped :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants