Add a step to check for required tools, fixes #46

ddev / ddev-platformsh

Add integration with Platform.sh hosting service

Apache License 2.0

9 stars 10 forks source link

Closed rfay closed 1 year ago

rfay commented 1 year ago

Current known local-install tools are base64, jq, perl;; this checks for them and warns that they're needed.

We'd like to get rid of external dependencies, but this will do for now.

https://github.com/drud/ddev/pull/4441 also improves this experience, by not showing a whole wall of text by default when a step fails.

lolautruche commented 1 year ago

We should probably add a requirement note in the README as well

ptmkenny commented 1 year ago

I just got bit by this today, and I think it'd be nice to have a (very) brief note in the README as well listing the dependencies.

lolautruche commented 1 year ago

Thanks @ptmkenny , I added a mention to the dependencies in the README in #78

rfay commented 1 year ago

Thanks so much @ptmkenny , looking forward to hearing about your other experiences as this matures and becomes ready for more projects.