Skip to content

Add php 8.2-8.4 to ci#4

Open
paladox wants to merge 14 commits intomasterfrom
paladox-patch-1
Open

Add php 8.2-8.4 to ci#4
paladox wants to merge 14 commits intomasterfrom
paladox-patch-1

Conversation

@paladox
Copy link
Copy Markdown
Contributor

@paladox paladox commented Mar 29, 2026

No description provided.

@paladox paladox closed this Mar 29, 2026
@paladox
Copy link
Copy Markdown
Contributor Author

paladox commented Mar 29, 2026

We’ve stoped using this lib in smw

@paladox paladox deleted the paladox-patch-1 branch March 29, 2026 02:00
@paladox paladox restored the paladox-patch-1 branch March 29, 2026 11:45
@paladox paladox reopened this Mar 29, 2026
@paladox
Copy link
Copy Markdown
Contributor Author

paladox commented Mar 29, 2026

PHP 8.2 changes the order of ->options (ksort does).

8.0:

{"ONOI_HTTP_REQUEST_RESPONSECACHE_PREFIX":"foo:","ONOI_HTTP_REQUEST_RESPONSECACHE_TTL":60,"10002":"","19913":true}

vs

8.2

{"10002":"","19913":true,"ONOI_HTTP_REQUEST_RESPONSECACHE_PREFIX":"foo:","ONOI_HTTP_REQUEST_RESPONSECACHE_TTL":60}

So the hash changes, so had to adapt the tests.

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.

1 participant