Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
jfy133 authored Oct 9, 2024
1 parent 424bad9 commit b87f780
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
3 changes: 3 additions & 0 deletions assets/schema_input.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,13 @@
"id": {
"type": "string",
"pattern": "^\\S+$",
"uniqueItems": true,
"errorMessage": "Sequence reference name must be provided and cannot contain spaces",
"meta": ["id"]
},
"taxid": {
"type": "integer",
"uniqueItems": true,
"errorMessage": "Please provide a valid taxonomic ID in integer format",
"meta": ["taxid"]
},
Expand All @@ -30,6 +32,7 @@
"maxLength": 0
}
],
"uniqueItems": true,
"errorMessage": "FASTA file for nucleotide sequence cannot contain spaces and must have a valid FASTA extension (fasta, fna, fa, fas, faa), optionally gzipped",
"exists": true,
"format": "file-path"
Expand Down
1 change: 0 additions & 1 deletion docs/output.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ The directories listed below will be created in the results directory after the
The pipeline is built using [Nextflow](https://www.nextflow.io/) and processes data using the following steps:

- [MultiQC](#multiqc) - Aggregate report describing results and QC from the whole pipeline

- [Pipeline information](#pipeline-information) - Report metrics generated during the workflow execution
- [Bracken](#bracken) - Database files for Brakcen
- [Centrifuge](#centrifuge) - Database files for Centrifuge
Expand Down
3 changes: 1 addition & 2 deletions nextflow.config
Original file line number Diff line number Diff line change
Expand Up @@ -179,8 +179,7 @@ profiles {
includeConfig !System.getenv('NXF_OFFLINE') && params.custom_config_base ? "${params.custom_config_base}/nfcore_custom.config" : "/dev/null"

// Load nf-core/createtaxdb custom profiles from different institutions.
// TODO nf-core: Optionally, you can add a pipeline-specific nf-core config at https://github.com/nf-core/configs
// includeConfig !System.getenv('NXF_OFFLINE') && params.custom_config_base ? "${params.custom_config_base}/pipeline/createtaxdb.config" : "/dev/null"
includeConfig !System.getenv('NXF_OFFLINE') && params.custom_config_base ? "${params.custom_config_base}/pipeline/createtaxdb.config" : "/dev/null"
// Set default registry for Apptainer, Docker, Podman, Charliecloud and Singularity independent of -profile
// Will not be used unless Apptainer / Docker / Podman / Charliecloud / Singularity are enabled
// Set to your registry if you have a mirror of containers
Expand Down

0 comments on commit b87f780

Please sign in to comment.