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.

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

DEP Set minimum version of symfony/cache to 7.1.5 #11466

Merged

Conversation

emteknetnz
Copy link
Member

@emteknetnz emteknetnz commented Nov 14, 2024

Issue silverstripe/.github#328

Two commits, so don't squash merge

1

Require https://github.com/symfony/cache/releases/tag/v7.1.5 which has a fix to work with newer versions of redis

Fixes --prefer-lowest build https://github.com/silverstripe/silverstripe-framework/actions/runs/11826436315/job/32952439907

PHP Fatal error: Declaration of Symfony\Component\Cache\Traits\Redis6Proxy::hSet($key, $member, $value): Redis|int|false must be compatible with Redis::hSet(string $key, mixed ...$fields_and_vals): Redis|int|false in /home/runner/work/silverstripe-framework/silverstripe-framework/vendor/symfony/cache/Traits/Redis6Proxy.php on line 519

2

Fix DataProviders in unit test after merge-up from 5

@emteknetnz emteknetnz force-pushed the pulls/6/broken-builds branch from 387b4b7 to 859f4f7 Compare November 14, 2024 03:09
@emteknetnz emteknetnz changed the title DEP Set minimum version of symfony/cache to 7.1.4 DEP Require symfony/cache-contracts 3 Nov 14, 2024
@emteknetnz emteknetnz force-pushed the pulls/6/broken-builds branch from 859f4f7 to 24e2a91 Compare November 14, 2024 03:57
@emteknetnz emteknetnz changed the title DEP Require symfony/cache-contracts 3 DEP Set minimum version of symfony/cache to 7.1.4 Nov 14, 2024
@emteknetnz emteknetnz changed the title DEP Set minimum version of symfony/cache to 7.1.4 DEP Set minimum version of symfony/cache to 7.1.7 Nov 14, 2024
@emteknetnz emteknetnz force-pushed the pulls/6/broken-builds branch 2 times, most recently from ab82621 to 3d868ce Compare November 14, 2024 04:12
@emteknetnz emteknetnz changed the title DEP Set minimum version of symfony/cache to 7.1.7 DEP Set minimum version of symfony/cache to 7.1.5 Nov 14, 2024
@emteknetnz emteknetnz force-pushed the pulls/6/broken-builds branch from 3d868ce to a08a119 Compare November 14, 2024 04:17
@emteknetnz emteknetnz marked this pull request as ready for review November 14, 2024 22:20
Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM

@GuySartorelli GuySartorelli merged commit 2830c87 into silverstripe:6 Nov 14, 2024
11 checks passed
@GuySartorelli GuySartorelli deleted the pulls/6/broken-builds branch November 14, 2024 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants