Skip to content

Conversation

@jasontedor
Copy link
Member

The documentation was missing the long option for the force option, and the short option for the stdin option. This commit addresses this by adding these to the documentation.

Relates #54229

The documentation was missing the long option for the force option, and
the short option for the stdin option. This commit addresses this by
adding these to the documentation.
@jasontedor jasontedor added >docs General docs changes :Core/Infra/Core Core issues without another label v8.0.0 v7.7.0 v7.8.0 labels Mar 26, 2020
@jasontedor jasontedor requested a review from pugnascotia March 26, 2020 03:02
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Core)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@jasontedor jasontedor merged commit c120388 into elastic:master Mar 26, 2020
@jasontedor
Copy link
Member Author

Oops, I did not mean to merge this without a review. Sorry. 🤦‍♀️

jasontedor added a commit that referenced this pull request Mar 26, 2020
The documentation was missing the long option for the force option, and
the short option for the stdin option. This commit addresses this by
adding these to the documentation.
jasontedor added a commit that referenced this pull request Mar 26, 2020
The documentation was missing the long option for the force option, and
the short option for the stdin option. This commit addresses this by
adding these to the documentation.
@jasontedor jasontedor deleted the document-keystore-cli-options branch March 26, 2020 04:02
Copy link
Contributor

@pugnascotia pugnascotia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM anyway!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >docs General docs changes v7.7.0 v7.8.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants