Require all tests to pass before packaging

[rkm]

Proposed Changes

AP forces you to manually check if the dependsOn tasks have succeeded otherwise it will always continue.

Types of changes

What types of changes does your code introduce? Tick all that apply.

• Bugfix (non-breaking change which fixes an issue)
• New Feature (non-breaking change which adds functionality)
• Breaking Change (fix or feature that would cause existing functionality to not work as expected)
• Documentation-Only Update (if none of the other choices apply)
  • In this case, ensure that the message of the head commit from the source branch is prefixed with [skip ci]

Checklist

By opening this PR, I confirm that I have:

• Reviewed the contributing guidelines for this repository
• Ensured that the PR branch is in sync with the target branch (i.e. it is automatically merge-able)
• Updated any relevant API documentation
• Created or updated any tests if relevant
• Created a news file
  • NOTE: This must include any changes to any of the following files: default.yaml, any of the RabbitMQ server configurations, GlobalOptions.cs
• Listed myself in the CONTRIBUTORS file 🚀
• Requested a review by one of the repository maintainers

Issues

[coveralls]

Coverage remained the same at 54.12% when pulling 6930a8b on fix/package-conditional into 707f262 on master.

[tznind]

Does this effectively change the CI to only release the package if all tests pass? That seems sensible but doesn't match the changelog/news/PR title. Maybe "Changed package release CI task to require all tests to pass"?

[rkm]

True - the fix was mainly that I had always intended that to be the case! Will update the docs