-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
3.0 RFC: Console Folder structure #4347
Comments
I would probably still leave |
I think they have to change it anyway to one of the above, regarding the different root folder names. |
We could also look at using composer to create |
Or that. But Symfony2 for example has a root bin dir and
Which makes sense as it reduces typing ( |
Would using both the composer config option in cakephp/app and having CakePHP export the bin script be a good first step? I have a small concern about switching directories around post beta, but if others are in favor I think it could work. |
👍 to the proposed directory and composer config changes. |
👍 to |
now that we are moving to namings like "src" shudder "bin" really makes sense (consistency) |
@ionas |
Closing as the pull request is merged. |
Moving the |
My mistake. |
I can take care of this unless someone else wants to. Feel free to assign to me if you wish. |
@bcrowe Steal it, steal it. @dereuromark doesn't seem to have come out of holiday mood yet 😄 |
True true |
Closing as cakephp/app#123 has been merged. |
1. bin/cake
First of all, the (remaining) non-class files should be removed from /src/Console.
Since
$ src/Console/cake [command]
or$ cd src && Console/cake [command]
is quite long it would make sense to put those files intobin
- as other projects do, as well.So it is just
$ bin/cake [command]
then.2. Console/Command => Shell
Secondly, the Console folder does not need to be that nested anymore.
We can move Command up, so it becomes a subfolder to app.
We should also rename it then, as we usually always have classes in a folder with the name of the type.
I recommend "Shell".
This simplifies the whole console part:
The text was updated successfully, but these errors were encountered: