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.

Sign up for GitHub

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

Jump to bottom

Fix panic when updating array with dynamic value #1606

Merged
merged 1 commit into from
Jun 3, 2024
Merged

Fix panic when updating array with dynamic value #1606

merged 1 commit into from
Jun 3, 2024

Commits on Jun 3, 2024

  1. Fix panic when updating array with dynamic value 

    This fixes a panic that was introduced in #1571, where if the content being used to update an array was a dynamic, non-array value it would panic when trying to aggreate a default array type into a element type. To make sure the update matches the chosen type, generate the `ValueKind` from the matching expression type. Added unit tests for both array and non-array dynamic update types.
    @swernli
    swernli committed Jun 3, 2024
    Configuration menu
    Copy the full SHA
    157500f View commit details
    Browse the repository at this point in the history