Skip to content

Add check arguments - #1928

Merged
mmatera merged 6 commits into
masterfrom
add-CheckArguments
Sep 2, 2026
Merged

Add check arguments#1928
mmatera merged 6 commits into
masterfrom
add-CheckArguments

Conversation

@rocky

@rocky rocky commented Sep 1, 2026

Copy link
Copy Markdown
Member

Add CheckArguments[].

CheckArguments is used in Rubi.

The association option "OptionsMode" and possibly the tag used in some failure messages (which change depending on whether an option was defined or how many arguments are given) may be different too.

However, this is good enough for Rubi (the infinite recursion seen before is now gone).

@rocky
rocky marked this pull request as draft September 1, 2026 01:47
@rocky
rocky force-pushed the add-CheckArguments branch 3 times, most recently from 42ed799 to 2e09d65 Compare September 1, 2026 09:24
@rocky
rocky force-pushed the add-CheckArguments branch 2 times, most recently from 7dae0eb to 15b4921 Compare September 1, 2026 09:54
@rocky
rocky force-pushed the add-CheckArguments branch from 15b4921 to 2475b8e Compare September 1, 2026 10:05
@rocky
rocky marked this pull request as ready for review September 1, 2026 18:11
@rocky
rocky requested a review from mmatera September 1, 2026 18:17

@mmatera mmatera left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mmatera
mmatera merged commit 17e1511 into master Sep 2, 2026
15 checks passed
@mmatera
mmatera deleted the add-CheckArguments branch September 2, 2026 18:20
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.

2 participants