Add description and license to packages#204
Conversation
Updates the `package.json` files for `@youversion/platform-core`, `@youversion/platform-react-hooks`, and `@youversion/platform-react-ui` to include a `description` and `license` field. This provides clearer context for each package.
|
Greptile SummaryThis PR adds Key observations:
Confidence Score: 5/5Safe to merge — changes are metadata-only with no runtime impact. All findings are P2 style suggestions. The core changes are correct: valid SPDX identifier, matching the root license, consistent versioning across packages. No functional, security, or behavioral issues are introduced. packages/core/package.json and packages/hooks/package.json — consider adding LICENSE files to their directories for complete npm publishing compliance. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[npm publish] --> B{package has files field?}
B -- Yes --> C[packages/ui\nfiles: dist, README.md, LICENSE]
B -- No --> D[packages/core\nno files field → publish all]
B -- No --> E[packages/hooks\nno files field → publish all]
C --> F{LICENSE file exists\nin package dir?}
D --> G{LICENSE file exists\nin package dir?}
E --> H{LICENSE file exists\nin package dir?}
F -- No --> I[pre-existing gap]
G -- No --> J[license text not bundled]
H -- No --> K[license text not bundled]
style J fill:#ffe0b2
style K fill:#ffe0b2
Reviews (2): Last reviewed commit: "Merge branch 'main' into cp/demo-github-..." | Re-trigger Greptile |
Updates the
package.jsonfiles for@youversion/platform-core,@youversion/platform-react-hooks, and@youversion/platform-react-uito include adescriptionandlicensefield. This provides clearer context for each package.