Skip to content

Fix array_reverse type for list{0: int, 1?: int} etc.#11150

Merged
orklah merged 3 commits intovimeo:6.xfrom
MoonE:fix-array-reverse-of-list--with-possibly-undefined-values
Feb 21, 2025
Merged

Fix array_reverse type for list{0: int, 1?: int} etc.#11150
orklah merged 3 commits intovimeo:6.xfrom
MoonE:fix-array-reverse-of-list--with-possibly-undefined-values

Conversation

@MoonE
Copy link
Contributor

@MoonE MoonE commented Nov 6, 2024

Fix #11148

I'm not entirely sure if $first_arg_array->properties can have string keys? The type suggests it but not sure if it is possible here, because currently those would be lost with my implementation.

@MoonE MoonE force-pushed the fix-array-reverse-of-list--with-possibly-undefined-values branch from c297b4c to dca758d Compare November 10, 2024 17:18
@orklah
Copy link
Collaborator

orklah commented Feb 16, 2025

Please rebase on 6.x so we can merge it :)

@MoonE MoonE changed the base branch from 5.x to 6.x February 16, 2025 19:12
@MoonE MoonE force-pushed the fix-array-reverse-of-list--with-possibly-undefined-values branch from dca758d to 713c1d1 Compare February 16, 2025 19:13
@orklah orklah added the release:fix The PR will be included in 'Fixes' section of the release notes label Feb 21, 2025
@orklah
Copy link
Collaborator

orklah commented Feb 21, 2025

Thanks!

@orklah orklah merged commit c929241 into vimeo:6.x Feb 21, 2025
8 of 9 checks passed
@MoonE MoonE deleted the fix-array-reverse-of-list--with-possibly-undefined-values branch February 21, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:fix The PR will be included in 'Fixes' section of the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

array_reverse on variable length list gives strange results

2 participants