docs - Add helper-get-function-arity readme [skip ci] #6532
Conversation
athomann
changed the title
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/5451/ |
1 similar comment
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/5451/ |
| @@ -1,5 +1,19 @@ | |||
| # @babel/helper-get-function-arity | |||
|
|
|||
| Function that returns the number of arguments or operands that a function takes. | |||
There was a problem hiding this comment.
Im not sure what did u mean by operands here. Would be good to clarify it, but I think saying that it returns number of arguments is quite enough + clarification on what is considered to be an argument, for that we can just send readers to https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/length
There was a problem hiding this comment.
Sounds good. The operands part was taken from the wikipedia definition. Will make changes.
| return false; | ||
| } | ||
|
|
||
| const argumentLength = getFunctionArity(method); |
There was a problem hiding this comment.
i would call it argumentsLength (plural)
xtuc
added
the
PR: Docs 📝
lock
bot
added
the
outdated
Add to the README for helper-get-function-arity
[skip ci]