cloudfoundry / cf-deployment-concourse-tasks

Apache License 2.0
23 stars 76 forks source link

Combine both run-cats tasks into one #99

Closed davewalter closed 4 years ago

davewalter commented 4 years ago

What is this change about?

Combine the functionality from the run-cats-with-provided-cli task into the run-cats task.

Please provide contextual information.

It seemed unnecessary to have two tasks. A symbolic link has been included to avoid breaking users of the old run-cats-with-provided-cli task, but that could be omitted if we want to release a major change instead.

Please check all that apply for this PR:

Did you update the README as appropriate for this change?

How should this change be described in release notes?

The run-cats task now accepts an optional cf-cli input to allow you to provide your own CF CLI binary to run CATs. This functionality was previously provided by the separate run-cats-with-provided-cli task, which has been deprecated in favor of this change.

What is the level of urgency for publishing this change?

Tag your pair, your PM, and/or team!

cf-gitbot commented 4 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/169356747

The labels on this github issue will be updated when the story is started.

linux-foundation-easycla[bot] commented 4 years ago

CLA Check
The committers are authorized under a signed CLA.