search

cloudfoundry / bosh-deployment-resource

Apache License 2.0
12 stars 26 forks source link

Cannot delete deployment using example from README as manifest parameter is required #34

Closed rmeharg closed 6 years ago

rmeharg commented 6 years ago

Should this parameter be optional as it is not needed to delete the deployment?

Or should the README example be updated to include the manifest parameter?

drich10 commented 6 years ago

Hey there, sorry for the misleading example. In the future I may remove the requirement, but it required more work to make it conditionally optional on the out command. I have updated the README to be a better example for now.

drich10 commented 6 years ago

After some further evaluation and a little more caffeine I went ahead and made the change which is now available in 2.8.1+. Thanks for raising this!