Skip to content

docs: describe commandDir() parameters (#1540)#1613

Merged
bcoe merged 4 commits intoyargs:masterfrom
bennycode:patch-1
Jun 2, 2020
Merged

docs: describe commandDir() parameters (#1540)#1613
bcoe merged 4 commits intoyargs:masterfrom
bennycode:patch-1

Conversation

@bennycode
Copy link
Copy Markdown
Contributor

The commandDir option is explained in Advanced Topics but missing from the API list.

@bcoe
Copy link
Copy Markdown
Member

bcoe commented May 30, 2020

@bennyn thanks for the contribution 👍 I'm excited to document this.

One thing jumps out at me, the formatting is a bit off compared to how we've documented parameters elsewhere in the doc:

https://github.com/yargs/yargs/blob/master/docs/api.md#commandmodule

Where we've just used a paragraph per parameter, with the parameter starting in back-ticks, mind matching this formatting?

@bennycode
Copy link
Copy Markdown
Contributor Author

Hey @bcoe, I will update my PR matching your style guide!

@bennycode
Copy link
Copy Markdown
Contributor Author

@bcoe Style updated! 👍

@bcoe bcoe merged commit 791cc59 into yargs:master Jun 2, 2020
@bcoe
Copy link
Copy Markdown
Member

bcoe commented Jun 2, 2020

@bennyn thank you for the contribution \o/

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.

2 participants