Skip to content

Comments

add dockerfile for control-freec#349

Merged
osallou merged 3 commits intoBioContainers:masterfrom
DavyCats:control-freec
Nov 26, 2019
Merged

add dockerfile for control-freec#349
osallou merged 3 commits intoBioContainers:masterfrom
DavyCats:control-freec

Conversation

@DavyCats
Copy link
Contributor

@DavyCats DavyCats commented Nov 25, 2019

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 container which is automatically built from bioconda lacks any fonts. As such the plots
    generated inside the container display only squares instead of actual text. Hence why this
    container is needed.

  • 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

@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

about.license field is not in spdx list: https://spdx.org/licenses/, if it is a typo error, please fix it. If this is not a standard license, then ignore this message.

@biocontainers-bot
Copy link
Collaborator

No control-freec/11.5/test-cmds.txt (test file) present, skipping tests

@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

about.license field is not in spdx list: https://spdx.org/licenses/, if it is a typo error, please fix it. If this is not a standard license, then ignore this message.

@biocontainers-bot
Copy link
Collaborator

No control-freec/11.5/test-cmds.txt (test file) present, skipping tests

LABEL about.summary="Prediction of copy numbers and allelic content using deep-sequencing data"
LABEL about.home="http://boevalab.inf.ethz.ch/FREEC/index.html"
LABEL about.documentation="http://boevalab.inf.ethz.ch/FREEC/index.html"
LABEL extra.identifiers.biotools="FREEC"
Copy link
Contributor

Choose a reason for hiding this comment

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

please use freec instead of FREEC for biotools entry (lowercase case is the real identifier in biotools), cf https://bio.tools/freec

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed!

@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

about.license field is not in spdx list: https://spdx.org/licenses/, if it is a typo error, please fix it. If this is not a standard license, then ignore this message.

@biocontainers-bot
Copy link
Collaborator

No control-freec/11.5/test-cmds.txt (test file) present, skipping tests

@osallou osallou merged commit 411c14b into BioContainers:master Nov 26, 2019
@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

about.license field is not in spdx list: https://spdx.org/licenses/, if it is a typo error, please fix it. If this is not a standard license, then ignore this message.

@biocontainers-bot
Copy link
Collaborator

No control-freec/11.5/test-cmds.txt (test file) present, skipping tests

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