buggood first issuemaths
Repository metrics
- Stars
- (48,709 stars)
- PR merge metrics
- (Avg merge 20d 6h) (157 merged PRs in 30d)
Description
The argument-handling of min, max, and minmax is currently somewhat inconsistent:
-
minandmaxcan have 1+ arguments (code for 3+ args here), butminmaxcan only have 1 or 2 arguments (from discourse) - all three can have 1 numeric argument, but this is not mentioned in the docstrings
-
minandmaxsupportmissing(returningmissing), but throw aMethodErrorwith a singlemissingargument alamin(missing) -
minmax(missing, 1)is incorrect, returning(1, missing)rather than(missing, missing)— this is a clear bug
These are all easy to fix, so marking as "good first issue". Feel free to submit a single PR for just one of the above items.