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

Clarify how to specify multiple arguments #15

Merged
merged 1 commit into from Apr 7, 2015
Merged

Clarify how to specify multiple arguments #15

merged 1 commit into from Apr 7, 2015

Conversation

austinpray
Copy link
Contributor

Hey there!

I personally don't have this issue in my understanding but I have noticed my users do. I keep seeing threads like this.

Let me know if this example is appropriate.

Thanks!

Edit:
Also I noticed that the MDN docs and elsewhere use this format for repeating arguments:

fun.bind(thisArg[, arg1[, arg2[, ...]]])

rather than

lazypipe().pipe(fn, [args...])

I have had people on my team comment that [args...] looks like a JS array. However I am used to seeing [args...] format from looking at man pages.
screenshot 2015-04-07 13 04 23

@JulienMelissas
Copy link

👍

1 similar comment
@kalenjohnson
Copy link

👍

@fgilio
Copy link

fgilio commented Apr 7, 2015

👍 thanks!

@OverZealous
Copy link
Owner

I'm fine if you want to update the [args...] for this PR as well. Otherwise, thanks for the PR!

@austinpray
Copy link
Contributor Author

@OverZealous updated!

OverZealous added a commit that referenced this pull request Apr 7, 2015
Clarify how to specify multiple arguments
@OverZealous OverZealous merged commit c479a62 into OverZealous:master Apr 7, 2015
@OverZealous
Copy link
Owner

Great, thanks! It's merged in. I probably won't make a release just for the doc changes, yet, unless it seems vital to everyone.

@OverZealous
Copy link
Owner

Actually, I changed my mind, it's also released as 0.2.3. :-)

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

Successfully merging this pull request may close these issues.

None yet

5 participants