Skip to content

Rastair #604

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

Merged
merged 3 commits into from
Aug 22, 2025
Merged

Rastair #604

merged 3 commits into from
Aug 22, 2025

Conversation

andrew-watchmaker
Copy link
Contributor

@andrew-watchmaker andrew-watchmaker commented Aug 21, 2025

Submitting a Container

(If you're requesting for a new container, please check the procedure described here.

Check BioContainers' Dockerfile specifications

Checklist

  1. Misc
  • My tool doesn't exist in BioConda
  • The image can be built
  1. Metadata
  • LABEL base_image
  • LABEL version
  • LABEL software.version
  • LABEL about.summary
  • LABEL about.home
  • LABEL about.license
  • MAINTAINER
  1. Extra (optionals)
  • I have written tests in test-cmds.txt
  • LABEL extra.identifier
  • LABEL about.documentation
  • LABEL about.license_file
  • LABEL about.tags

Check BioContainers' Dockerfile metadata

@andrew-watchmaker
Copy link
Contributor Author

PR for Issue #603

@biocontainers-bot
Copy link
Collaborator

software.version label not matching directory version name

@biocontainers-bot
Copy link
Collaborator

please specify in about.license_file the location of the license file in the container, or a url to license for this release of the software.

@biocontainers-bot
Copy link
Collaborator

No biotools label defined, please check if tool is not already defined in biotools (https://bio.tools) and add extra.identifiers.biotools label if it exists. If it is not defined, you can ignore this comment.

@mboudet
Copy link
Contributor

mboudet commented Aug 22, 2025

Thanks for the PR. The only blocking error is the 'software.version label not matching directory version name'. The directory containing the Dockerfile should match the software version (so v0.8.2).

The license_file warning is non-blocking, since there is a clear LICENSE file in the remote.

@andrew-watchmaker
Copy link
Contributor Author

Thanks for that @mboudet! Just pushed the directory rename

@biocontainers-bot
Copy link
Collaborator

please specify in about.license_file the location of the license file in the container, or a url to license for this release of the software.

@biocontainers-bot
Copy link
Collaborator

No biotools label defined, please check if tool is not already defined in biotools (https://bio.tools) and add extra.identifiers.biotools label if it exists. If it is not defined, you can ignore this comment.

@biocontainers-bot
Copy link
Collaborator

No test-cmds.txt (test file) present, skipping tests

@mboudet mboudet merged commit d413b50 into BioContainers:master Aug 22, 2025
1 check passed
@andrew-watchmaker andrew-watchmaker deleted the rastair branch August 22, 2025 21:11
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.

3 participants