Force gha to pass

[adamrtalbot]

PR checklist

Closes #XXX

• This comment contains a description of changes (with reason).
• If you've fixed a bug or added code that should be tested, add tests!
• If you've added a new tool - have you followed the module conventions in the contribution docs
• If necessary, include test data in your PR.
• Remove all TODO statements.
• Emit the versions.yml file.
• Follow the naming conventions.
• Follow the parameters requirements.
• Follow the input/output options guidelines.
• Add a resource label
• Use BioConda and BioContainers if possible to fulfil software requirements.
• Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
  • PROFILE=docker pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
  • PROFILE=singularity pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
  • PROFILE=conda pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware

[adamrtalbot]

Branch protection rules:

[drpatelh]

Suggested change

	run-name: nf-core linting
	run-name: nf-core lint

[drpatelh]

Suggested change

	name: check-for-changes
	name: check-changes

[adamrtalbot]

You can't just change names because you don't like how they look 😉 These matter for the branch protection.

[drpatelh]

Suggested change

	name: check-for-changes
	name: check-changes

[drpatelh]

Suggested change

	name: check-for-changes
	name: check-changes

[drpatelh]

Suggested change

	run-name: pytest-workflow
	run-name: pytest

[drpatelh]

Suggested change

	pytest_workflow:
	pytest:

[drpatelh]

Suggested change

	name: pytest-workflow
	name: pytest

[drpatelh]

Suggested change

	nf_test:
	nf-test:

[drpatelh]

Suggested change

	name: code-linting
	name: code-lint

[drpatelh]

Suggested change

	fix_linting:
	fix-linting:

[drpatelh]

Excellent spot 🚀 I suggested some minimised names for the CI tests just so that we don't go off screen when we include module names at the end.

[adamrtalbot]

@drpatelh go review nf-core#3677 if you want to actually check stuff.

[adamrtalbot]

Excellent spot 🚀 I suggested some minimised names for the CI tests just so that we don't go off screen when we include module names at the end.

The test names are the IDs used for branch protection, so they matter in some circumstances, although they can be changed of course.

[drpatelh]

@drpatelh go review nf-core#3677 if you want to actually check stuff.

Done!

The test names are the IDs used for branch protection, so they matter in some circumstances, although they can be changed of course.

Yep, since we are making the changes now is a good time as any.