Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.Sign up
assert: validate required arguments #26641
This validates most
I defensively marked this as semver-major while I guess it could also be
So if people are fine with this being a patch instead, I am happy to remove the label.
Trott left a comment
LGTM other than the very minor change that I think some tests with the functions getting zero arguments should be added. (Currently,
I can see it being patch and won't block that if that's what everyone else thinks, but I think
Once those tests are added, feel free to re-request a review and I'll change to "approve".
sam-github left a comment