search

pivotal / docs-platform-automation

docs repo for the Platform Automation package
https://docs.pivotal.io/platform-automation/
Other
13 stars 12 forks source link

Update task.md #19

Closed toomeykh closed 5 years ago

toomeykh commented 5 years ago

The page says that the stemcell download is optional but doesn't say how to enable/disable.

pivotal-issuemaster commented 5 years ago

@toomeykh Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

cf-gitbot commented 5 years ago

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

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

pivotal-issuemaster commented 5 years ago

@toomeykh Thank you for signing the Contributor License Agreement!

anEXPer commented 5 years ago

Thanks for this PR!

We'll probably take and then modify it. We appreciate the need to call attention to and describe this behavior, but the list of valid IaaS names and the description of how it works aren't quite right.

I'd expect to see us get to this in the next day or two.

anEXPer commented 5 years ago

@toomeykh we pulled this in and reworked it a bit. Let us know what you think; it should be live on our staging environment shortly.

toomeykh commented 5 years ago

@anEXPer That's a definite improvement over my attempt. Great catch on the "if it's a .pivotal file", I totally missed that. Thanks for the help with my first public commit!