Skip to content

TECH-15245: alias unpack to table.unpack#14

Closed
hislopzach wants to merge 1 commit intomasterfrom
TECH-15245_alias_unpack_to_table_unpack
Closed

TECH-15245: alias unpack to table.unpack#14
hislopzach wants to merge 1 commit intomasterfrom
TECH-15245_alias_unpack_to_table_unpack

Conversation

@hislopzach
Copy link
Contributor

Adding an alias for unpack to table.unpack to ensure compatability for scripts written for lua 5.1 that are now being run on 5.4. Impetus for this change is test failures for Graphql Pro subscriptions in my web passion project (https://buildkite.com/invoca/web/builds/97869#0195ab45-a052-41d4-b039-d868b0815148)

From Lua 5.2 reference manual

Function unpack was moved into the table library and therefore must be called as table.unpack.

@hislopzach hislopzach closed this Apr 17, 2025
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