10000 Remove restriction on the type of require results. by aatxe · Pull Request #227 · luau-lang/lute · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Remove restriction on the type of require results. #227

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.

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

Merged
merged 2 commits into from
Apr 29, 2025

Conversation

aatxe
Copy link
Collaborator
@aatxe aatxe commented Apr 29, 2025

We originally implemented a restriction on require results being a table or function so that scripts would intrinsically be forwards-compatible as they evolve. To my surprise, this is a controversial choice. As part of the as-of-yet-unwritten style guide of Lute, we'll never return anything that is not a table or function, but users are free to do how they see fit.

@Dekkonot

This comment was marked as resolved.

@aatxe aatxe changed the title People would rather require scripts that aren't forwards-compatible with extension. Remove restriction on require results. Apr 29, 2025
@aatxe aatxe changed the title Remove restriction on require results. Remove restriction on the type of require results. Apr 29, 2025
@aatxe aatxe enabled auto-merge (squash) April 29, 2025 00:15
@aatxe aatxe merged commit 7e88715 into primary Apr 29, 2025
6 checks passed
@aatxe aatxe deleted the remove-type-restriction-on-require-results branch April 29, 2025 00:19
green-real pushed a commit to green-real/lute that referenced this pull request May 19, 2025
We originally implemented a restriction on require results being a table
or function so that scripts would intrinsically be forwards-compatible
as they evolve. To my surprise, this is a controversial choice. As part
of the as-of-yet-unwritten style guide of Lute, we'll never return
anything that is not a table or function, but users are free to do how
they see fit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0