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.

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

Argument names #5

Closed
Richienb opened this issue Jan 17, 2024 · 2 comments
Closed

Argument names #5

Richienb opened this issue Jan 17, 2024 · 2 comments

Comments

@Richienb
Copy link
Collaborator

Richienb commented Jan 17, 2024

#3 (comment)

The specification uses highest and lowest: tc39.es/ecma262/multipage/numbers-and-dates.html#sec-math.max

Meanwhile val/value/num/number, min/minimum and max/maximum are commonplace.

@Richienb
Copy link
Collaborator Author

Richienb commented Jan 17, 2024

If we are to adopt options object argument, I prefer the shorter min and max over the longer names because they would take longer to type without helping that much in readability.

@Richienb
Copy link
Collaborator Author

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

No branches or pull requests

1 participant