-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Improve packages docs - document root package json and public API knowledge #31598
Improve packages docs - document root package json and public API knowledge #31598
Conversation
Size Change: -59 B (0%) Total Size: 1.31 MB
ℹ️ View Unchanged
|
packages/README.md
Outdated
@@ -115,6 +117,10 @@ npm install glob --save-dev | |||
|
|||
This commands adds the latest version of `glob` as a development dependency to the `package.json` file. It has to be executed from the root of the project. | |||
|
|||
## Maintaining a Public API |
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.
It looks like you could put this paragraph under the next section which is also about API 😄
We can also create a connection with the next paragraph but putting emphasis that generated API helps to find out what's actually becomes Public API.
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.
Nice one. It should help prevent the issues you had today.
I proposed a small change to the second paragraph added, but it's a minor thing.
Description
@gziolo provided some excellent advice around:
package.json
.This PR improves the documentation to capture this knowledge.
How has this been tested?
Reading words and checking they are aligned with @gziolo and his thoughts on packages.
Screenshots
Types of changes
New feature (non-breaking change which adds functionality)
Checklist:
*.native.js
files for terms that need renaming or removal).