-
-
Notifications
You must be signed in to change notification settings - Fork 111
refactor(core): move internal storage to project root #1285
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
Conversation
Why not |
@erikaraujo That doesn't solve the issue I have with |
@innocenzi The reason I like my suggestion is that it clearly states that it is a framework-specific directory and then it has a more specific "internal" directory inside of it. Or maybe do |
@innocenzi - I think the question I would ask is whether there are going to be other items we need to store (e.g., IMO, I think Laravel gets this fairly spot on in their app structure. |
I'm not sure I understand what you're asking. Local storage would be in |
I'm of the mind While I can appreciate the concept of a hidden directory, I can just imagine the number of people trying to figure out where the files are because they didn't list hidden directories, etc. |
As far as I know, PhpStorm, WebStorm and VSC all show dot files and directories. A |
Yeah I think |
Closes #1281
Before merging, is
.tempest
the ideal name? Could also bestorage
,.internal
, or something else. I like a dot directory because it's naturally ignored when scanning the hierarchy, but the name "tempest" is not generic and generally I don't like thatAlso, we need to add it to
.gitignore
intempestphp/tempest-app