-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Update cli-description #196208
Update cli-description #196208
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
self review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Just highlighting some Grammarly-driven suggestions 😇
@@ -19,8 +19,7 @@ const program = new Command('bin/kibana'); | |||
program | |||
.version(pkg.version) | |||
.description( | |||
'Kibana is an open and free, browser ' + | |||
'based analytics and search dashboard for Elasticsearch.' | |||
'Kibana is an open source, browser based analytics and search dashboard for Elasticsearch.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grammarly suggests these fixes:
'Kibana is an open source, browser based analytics and search dashboard for Elasticsearch.' | |
'Kibana is an open source browser-based analytics and search dashboard for Elasticsearch.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for catching this 🙂
💚 Build Succeeded
Metrics [docs]
History
|
Updates the
cli
description text to reflect license change from #192025.