Closed ypriverol closed 7 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 1ba5808
+| ✅ 278 tests passed |+
#| ❔ 4 tests were ignored |#
!| ❗ 1 tests had warnings |!
@fabianegli @jpfeuffer do you remember how to fix the error when building the PR for release, that the version do not match the comments:
╭─ [✗] 1 Pipeline Test Failed ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ │
│ multiqc_config: 'assets/multiqc_config.yml' does not contain a matching 'report_comment'. │
│ The expected comment is: │
│ This report has been generated by the <a href="https://github.com/nf-core/quantms/releases/tag/1.3.0" target="_blank">nf-core/quantms</a> analysis pipeline. For information about how to interpret these results, please see │
│ the <a href="https://nf-co.re/quantms/1.3.0/docs/output" target="_blank">documentation</a>. │
│ The current comment is: │
│ This report has been generated by the <a href="https://github.com/nf-core/quantms/tree/dev" target="_blank">nf-core/quantms</a> analysis pipeline. For information about how to interpret these results, please see the <a │
│ href="https://nf-co.re/quantms/dev/docs/output" target="_blank">documentation</a>.
@fabianegli @jpfeuffer do you remember how to fix the error when building the PR for release, that the version do not match the comments:
╭─ [✗] 1 Pipeline Test Failed ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮ │ │ │ multiqc_config: 'assets/multiqc_config.yml' does not contain a matching 'report_comment'. │ │ The expected comment is: │ │ This report has been generated by the <a href="https://github.com/nf-core/quantms/releases/tag/1.3.0" target="_blank">nf-core/quantms</a> analysis pipeline. For information about how to interpret these results, please see │ │ the <a href="https://nf-co.re/quantms/1.3.0/docs/output" target="_blank">documentation</a>. │ │ The current comment is: │ │ This report has been generated by the <a href="https://github.com/nf-core/quantms/tree/dev" target="_blank">nf-core/quantms</a> analysis pipeline. For information about how to interpret these results, please see the <a │ │ href="https://nf-co.re/quantms/dev/docs/output" target="_blank">documentation</a>.
did you run nf-core bump-version 1.3.0
?
@fabianegli @jpfeuffer do you remember how to fix the error when building the PR for release, that the version do not match the comments:
╭─ [✗] 1 Pipeline Test Failed ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮ │ │ │ multiqc_config: 'assets/multiqc_config.yml' does not contain a matching 'report_comment'. │ │ The expected comment is: │ │ This report has been generated by the <a href="https://github.com/nf-core/quantms/releases/tag/1.3.0" target="_blank">nf-core/quantms</a> analysis pipeline. For information about how to interpret these results, please see │ │ the <a href="https://nf-co.re/quantms/1.3.0/docs/output" target="_blank">documentation</a>. │ │ The current comment is: │ │ This report has been generated by the <a href="https://github.com/nf-core/quantms/tree/dev" target="_blank">nf-core/quantms</a> analysis pipeline. For information about how to interpret these results, please see the <a │ │ href="https://nf-co.re/quantms/dev/docs/output" target="_blank">documentation</a>.
did you run
nf-core bump-version 1.3.0
?
I did it manually.
@nf-core-bot fix linting
@mashehu I can't find that URL or mention of crisprseq in our README.md can you point me to the line.
@mashehu We have for the pipeline a lot of documentation including explanations about the topics and mass spectrometry, figures, etc is not only about the usage.
I don't see why this can't be on the website (gives the material also better discoverability on search engines). If it is needed/helps to understand how to use/interpret the pipeline, it should be on nf-co.re. We have pipelines like taxprofiler with extensive documentation. It also points to the wrong repository in the installation steps there (nextflow run bigbio/quantms
instead of nextflow run nf-core/quantms
).
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).