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

[Clarity] use stricter argument count checks #3117

Open
jcnelson opened this issue Apr 28, 2022 · 1 comment
Open

[Clarity] use stricter argument count checks #3117

jcnelson opened this issue Apr 28, 2022 · 1 comment
Assignees
@obycode
Labels
icebox Issues that are not being worked on

Comments

@jcnelson
Copy link
Member

In some functions like get-block-info? and var-set, we only require a minimum argument count as opposed to an exact argument count. I think we should require an exact argument count in 2.1, so that failure to supply the right number of arguments results in a CheckError instead of an InterpreterError.
@jcnelson jcnelson self-assigned this Apr 28, 2022
@jcnelson jcnelson mentioned this issue Apr 28, 2022
Merged
@jcnelson jcnelson added this to To do in Stacks 2.1 via automation Apr 28, 2022
@jcnelson
Copy link
Member Author

jcnelson commented May 2, 2022

This is a go

@jcnelson jcnelson removed this from To do in Stacks 2.1 May 23, 2022
@jcnelson jcnelson added the icebox Issues that are not being worked on label Mar 7, 2023
@jcnelson jcnelson assigned obycode and unassigned jcnelson Mar 7, 2023
@obycode obycode mentioned this issue Jun 26, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@obycode obycode

Labels
icebox Issues that are not being worked on
Projects
Stacks Core Eng
Status: Status: 🆕 New
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jcnelson @obycode @pavitthrap