ci: never skip the success step

[cole-h]

Description

Checklist

• Tested changes against a test repository
• Added or updated relevant documentation (leave unchecked if not applicable)
• (If this PR is for a release) Updated README to point to the new tag (leave unchecked if not applicable)

[grahamc]

If we want it to not run when the job was cancelled, we might want ...

Suggested change

	if: always()
	if: success() || failure()

[cole-h]

I don't want to think about how that would interact; it's safer to just say "always run this", since we'll only exit with a failure if the following if condition is satisfied.