Wrong links in "assets/multiqc_config.yml"

[drpatelh]

Description of the bug

The link rendered when the pipeline template below doesn't work:

tools/nf_core/pipeline-template/assets/multiqc_config.yml

Line 2 in e5ce6ce

This report has been generated by the <a href="https://github.com/{{ name }}/{{ version}}" target="_blank">{{ name }}</a>

I believe it should be href="https://github.com/{{ name }}/tree/{{ version}}"

Also, a related question. Will the link below work on the website if for example I have a dev version of a pipeline defined in manifest.version e.g. 3.13.0dev. Could be another bug there too.

tools/nf_core/pipeline-template/assets/multiqc_config.yml

Line 4 in e5ce6ce

<a href="https://nf-co.re/{{ short_name }}/{{ version }}/output" target="_blank">documentation</a>.{% endif %}

[maxulysse]

I had the dev in sarek, something is weird there

[SusiJo]

Is this wrong link related to this linting error?

[✗] 1 Pipeline Test Failed                                                                           
multiqc_config: 'assets/multiqc_config.yml' does not contain a matching 'report_comment'.                                                          

[SusiJo]

The linting of the multiqc_config

tools/nf_core/lint/multiqc_config.py

Line 74 in e5ce6ce

version = self.nf_config.get("manifest.version", "").strip(" '\"")

checks for the manifest.version in the nextflow.config. However, if the dev branch has a version string like manifest.version = '2.1.0dev'. There will not be any such tag available for creating the link described above.
So maybe the manifest.version should be taken from the nextflow.config of the master branch or the dev string removed.