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
[Docs] Adds clarification in user-guide documentation #4256
Conversation
it is not clear or documented anywhere that a local file path is an option as argument to the path parameter. cli _help_ scheme, as well, implies the last argument to be a url (`hint [options] https://url.com`). as only starting to use webhint cli I had this only clarified in a response in the gitter channel
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.
Thanks for putting this together @yinonov!
@@ -26,7 +26,7 @@ best practices that may be applied to your site, complete the following steps. | |||
uses `npx` to run the `npm` package without installing it. | |||
|
|||
```bash | |||
npx hint https://example.com | |||
npx hint https://example.com // or relative path e.g. './path/to/my.html' |
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.
What do you think about breaking these apart as two complete commands? I think that might help for those that want to copy-paste a local file example with minimal required changes. E.g.
npx hint https://example.com
or
npx hint ./path/to/my.html
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.
sure, it's nicer this way
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 helping improve the documentation!
I'll merge this after CI finishes, then the doc changes will go live with our next release.
Removes trailing spaces the linter was complaining about.
it is not clear or documented anywhere that a local file path is an option as argument to the path parameter.
cli help scheme, as well, implies the last argument to be a url (
hint [options] https://url.com
).as only starting to use webhint cli I had this only clarified in a response in the gitter channel
Pull request checklist
Make sure you:
For non-trivial changes, please make sure you also:
Short description of the change(s)