Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update MicroKernelTrait directory structure #6868

Closed
cadavre opened this issue Aug 11, 2016 · 0 comments
Closed

Update MicroKernelTrait directory structure #6868

cadavre opened this issue Aug 11, 2016 · 0 comments
Labels
hasPR A Pull Request has already been submitted for this issue.

Comments

@cadavre
Copy link

cadavre commented Aug 11, 2016

Micro kernel documentation (https://symfony.com/doc/current/configuration/micro_kernel_trait.html) states that it uses "traditional Symfony application" structure, but since Symfony 3 we have different default locations for logs/ and cache/ (https://symfony.com/doc/current/configuration/override_dir_structure.html).

We should add getLogDir() and getCacheDir() override examples to fit new directory structure.
@javiereguiluz javiereguiluz mentioned this issue Aug 31, 2016
Closed
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Aug 31, 2016
xabbuh added a commit that referenced this issue Sep 21, 2016
@xabbuh
minor #6935 Use the standard cache and logs dir for the micro kernel …
7be8b3f 
…example (javiereguiluz)

This PR was submitted for the 3.0 branch but it was merged into the 3.1 branch instead (closes #6935).

Discussion
----------

Use the standard cache and logs dir for the micro kernel example

This fixes #6868.

Commits
-------

ca53cb5 Use the standard cache and logs dir for the micro kernel example
@xabbuh xabbuh closed this as completed Sep 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@javiereguiluz @cadavre @xabbuh