buggood first issuemaths
Métricas do repositório
- Stars
- (48.709 stars)
- Métricas de merge de PR
- (Mesclagem média 23d 11h) (145 fundiu PRs em 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.