Skip to content

[Chat][Cache] Require generic PSR-6, instead of Symfony Cache#2095

Open
10n wants to merge 1 commit into
symfony:mainfrom
10n:patch-1
Open

[Chat][Cache] Require generic PSR-6, instead of Symfony Cache#2095
10n wants to merge 1 commit into
symfony:mainfrom
10n:patch-1

Conversation

@10n

@10n 10n commented May 18, 2026

Copy link
Copy Markdown

Added PSR cache requirement and suggestion.
The cache bridge is actually using generic Psr\Cache\CacheItemPoolInterface

Q A
Bug fix? yes
New feature? no
Docs? yes
Issues n/a
License MIT

Update component requirement to indicate the actual dependency to PSR-6 psr/cache, not Symfony Cache symfony/cache.
I used the same version that is required by the symfony/cache

Added PSR cache requirement and suggestion.
The cache bridge is actually using generic `Psr\Cache\CacheItemPoolInterface`
@10n 10n requested review from OskarStark and chr-hertel as code owners May 18, 2026 08:49
@carsonbot carsonbot added Bug Something isn't working Status: Needs Review labels May 18, 2026

@Guikingone Guikingone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch 👍🏻

@10n 10n changed the title [AI Chat - Cache Store] Require generic PSR-6, instead of Symfony Cache [Chat ][Cache] Require generic PSR-6, instead of Symfony Cache May 18, 2026
@10n 10n changed the title [Chat ][Cache] Require generic PSR-6, instead of Symfony Cache [Chat][Cache] Require generic PSR-6, instead of Symfony Cache May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working Status: Reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants