search

cloudfoundry / cf-deployment-concourse-tasks

Apache License 2.0
23 stars 76 forks source link

Correct VARS_STORE_FILE description #66

Closed anEXPer closed 6 years ago

anEXPer commented 6 years ago

A question in slack brought it to my attention that this description wasn't, perhaps, updated when the option to not use a vars store was added.

Here's the code for the behavior this change describes: https://github.com/cloudfoundry/cf-deployment-concourse-tasks/blob/afc0d171fd44543a138133b909e5375b4db8f44a/shared-functions#L103-L107

I've only corrected this here, other tasks with similar parameters may have the same issue.

cfdreddbot commented 6 years ago

Hey anEXPer!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

cf-gitbot commented 6 years ago

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

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

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

heyjcollins commented 6 years ago

Thank you @anEXPer!