Skip to content

Fix the type of the options of the RedisArray constructor - #6399

Open
mlocati wants to merge 1 commit into
phpstan:2.2.xfrom
mlocati-forks:redis-array-options-type
Open

Fix the type of the options of the RedisArray constructor#6399
mlocati wants to merge 1 commit into
phpstan:2.2.xfrom
mlocati-forks:redis-array-options-type

Conversation

@mlocati

@mlocati mlocati commented Sep 8, 2026

Copy link
Copy Markdown

The options may have any type: for example, connect_timeout and read_timeout are numbers, lazy_connect and consistent are booleans, function and distributor are callables, previous is an array.

See the phpredis documentation of RedisArray:
https://github.com/phpredis/phpredis/blob/develop/arrays.md
and the phpredis stub of the constructor:
https://github.com/phpredis/phpredis/blob/develop/redis_array.stub.php

@phpstan-bot

Copy link
Copy Markdown
Collaborator

You've opened the pull request against the latest branch 2.3.x. PHPStan 2.3 is not going to be released for months. If your code is relevant on 2.2.x and you want it to be released sooner, please rebase your pull request and change its target to 2.2.x.

@mlocati
mlocati changed the base branch from 2.3.x to 2.2.x September 8, 2026 14:58
The options may have any type: for example, connect_timeout and
read_timeout are numbers, lazy_connect and consistent are booleans,
function and distributor are callables, previous is an array.

See the phpredis documentation of RedisArray:
https://github.com/phpredis/phpredis/blob/develop/arrays.md
and the phpredis stub of the constructor:
https://github.com/phpredis/phpredis/blob/develop/redis_array.stub.php
@mlocati
mlocati force-pushed the redis-array-options-type branch from 93fa9f6 to 0af0a71 Compare September 8, 2026 14:59
@mlocati

mlocati commented Sep 8, 2026

Copy link
Copy Markdown
Author

please rebase your pull request and change its target to 2.2.x

Done.

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