-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[FrameworkBundle] Add support to easily clear all cache pools #49705
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
1bb7d23
to
9f2be24
Compare
quick reminder that new features require PR in symfony/symfony-docs as well ;) |
You're absolutely right, I should have mentioned I will make that addition when I at least know this is going to be accepted in some form. The issue I link might indicate that already, but as it was quite old I find it best to have confirmation first :) Test errors look to be unrelated by the way. |
src/Symfony/Bundle/FrameworkBundle/Command/CachePoolClearCommand.php
Outdated
Show resolved
Hide resolved
4caa884
to
3a50d2d
Compare
Thank you @bobvandevijver. |
…for `cache:pool:clear` (bobvandevijver) This PR was squashed before being merged into the 6.3 branch. Discussion ---------- [Cache] Add documentation for new `--all` option for `cache:pool:clear` <!-- If your pull request fixes a BUG, use the oldest maintained branch that contains the bug (see https://symfony.com/releases for the list of maintained branches). If your pull request documents a NEW FEATURE, use the same Symfony branch where the feature was introduced (and `6.x` for features of unreleased versions). --> Documentation update for symfony/symfony#49705. Commits ------- bb7796d [Cache] Add documentation for new `--all` option for `cache:pool:clear`
As suggested in #22047, I've now added a
--all
option to clear all cache pools by just providing that argument tocache:pool:clear
. I believe this is more convenient than remembering thecache.global_clearer
clearer alias and more descriptive for the consumer.