search

configuredco / haulable

Create standalone PHP CLI applications with PHP Micro CLI
MIT License
163 stars 3 forks source link

Add an option to provide the platform via the CLI #1

Closed thibodelanghe closed 1 year ago

thibodelanghe commented 1 year ago

I added that it is possible to define the platform via the CLI instead of an interactive form. I'm still not 100% satisfied with the naming of the platforms.

If anyone has suggestions, please let me know!

Meanwhile, views have also been added to keep the html out of the command so it's a little cleaner.

joecampo commented 1 year ago

Hey, thanks for the PR! I think this is a good option and appreciate your contribution. If you decide to PR anything else I just ask if you could kindly keep the PR scoped to the feature. I'm very particular in code style and won't merge anything with unnecessary refactoring (I'm keeping my one liner collections! 😅).

thibodelanghe commented 1 year ago

Thanks @joecampo! I will keep an eye on it. Got a little carried away with the excitement 😅