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
refactor: project documentation #140
Conversation
readme.md is broken into api.md, example.md and readme.md
reorder according to alphabetical order mostly
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.
Looks great! Just added some minor changes.
Co-authored-by: Evan Shortiss <evanshortiss@gmail.com>
@@ -265,8 +265,7 @@ String. It's highly unlikely that a variable will not be a String since all | |||
#### asUrlObject() | |||
|
|||
Verifies that the variable is a valid URL string using the same method as | |||
`asUrlString()`, but instead returns the resulting URL instance. For details | |||
see the [Node.js URL docs](https://nodejs.org/docs/latest/api/url.html#url_class_url). | |||
`asUrlString()`, but instead returns the resulting URL instance. It uses the WHATWG URL constructor. |
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.
@evanshortiss did not accept your suggestion for this line because it would have resulted in the text becoming 'For details WHATWG URL constructor.'
Changed it to 'It uses the WHATWG URL constructor.'
Would this work?
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.
Looks good. I knew I'd mess up at least one suggestion 😄
Resolves #135 .
Changelog:
API.md
andEXAMPLE.md
- extract API section and secondary examples fromREADME.md
into these filesfrom()
link in API sectionLet me know what you think @evanshortiss ! You can preview this refactor at https://github.com/DigiPie/env-var/tree/docs/example