-
Notifications
You must be signed in to change notification settings - Fork 17
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
Updating vip cache commands descriptions and examples to follow the VIP-CLI style guide #1877
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Manifest Files |
Quality Gate passedIssues Measures |
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.
Tested with:
./dist/bin/vip-cache-purge-url.js -h
./dist/bin/vip-cache.js -h
usage: | ||
`${ exampleUsage } https://example-app-develop.go-vip.co/sample-page/` + | ||
'\n - Purged URL: https://example-app.develop.go-vip.co/sample-page/', | ||
description: 'Purge the page cache for a single URL.', |
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.
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.
I just tested this and can confirm that the example output matches the actual output when that command is run successfully. 👍
LGTM. Thanks for the careful review, @sanmai ! |
Updated the voice, tense, and format of descriptions and examples for the
vip cache
commands to follow the VIP-CLI style guide.Pull request checklist
New release checklist
Steps to Test
Observe the results of the
--help
display for impacted commands, such as:vip --help
vip cache --help
vip cache purge-url --help