search

svenluijten / forge-cli

🔥 A command line utility to interact with your Laravel Forge servers, sites, and more.
https://svenluijten.com
MIT License
64 stars 14 forks source link

Rename commands to `class:action` #26

Closed thoresuenert closed 4 years ago

thoresuenert commented 5 years ago

The command output is hard to scan. Does it make sense to go with the following for example:

instead of 

install
 install:certificate
 install:git

go with

certificate
  certificate:install
  certificate:list

for creating things we can go the laravel route and prefix it with make

make
  make:site
  make:certificate

Just a start for a conversation, i am not sure right now which could be the best way to sort stuff. The confusing stuff for me is some commands are only for servers others for sites etc.

svenluijten commented 5 years ago

Yeah, this is definitely something I struggled with when setting up the package initially. I do agree that it's hard to scan as it is now, but I wanted to keep it "Laravel-y", so went with action:resource (in line with make:controller etc.).

It wouldn't make sense (in my opinion) to add make:site, and then site:deploy or something, this would fragment the whole setup. I guess we could make it a configuration option somehow? 🤔

thoresuenert commented 5 years ago

@svenluijten i would argue against :D

laravel-ish would be:

make
  make:migration

migration
  migration:fresh
  migration:install