Skip to content

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

Merged
orklah merged 3 commits into
vimeo: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 into
vimeo:6.xfrom
MoonE:fix-array-reverse-of-list--with-possibly-undefined-values

Conversation

@MoonE

@MoonE MoonE commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

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

orklah commented Feb 16, 2025

Copy link
Copy Markdown
Collaborator

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

orklah commented Feb 21, 2025

Copy link
Copy Markdown
Collaborator

Thanks!

@orklah
orklah merged commit c929241 into vimeo:6.x Feb 21, 2025
@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