chore: add homepage and bugs metadata#385
Conversation
📝 WalkthroughWalkthroughThis PR updates ChangesPackage Metadata
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
f69695c to
8b5e808
Compare
|
The previous commit was authored by an unlinked |
|
@CLAassistant — I have signed the Contributor License Agreement via https://cla-assistant.io/OneBusAway/js-sdk?pullRequest=385 using the @codeboost-tr GitHub account. Please re-run the CLA check on this PR. The author email mismatch (test@example.com) was also addressed via a forced amend — the latest commits are authored by the GitHub-verified email. |
|
Bumping — has the CLA check run again after the author-email amend? The original CLA failure was on the unverified |
What
Adds the standard npm package metadata fields (
homepage,bugs) topackage.jsonso the package follows the npm registry conventions andshows up correctly in npmjs.com and the npm CLI.
Why
Closes the consistency gap with the rest of the OBA monorepo packages
which already include these fields. Also resolves the "metadata
warning" reported by
npm publish --dry-runfor this package.Changes
package.json:"homepage": "https://github.com/OneBusAway/js-sdk#readme""bugs": { "url": "https://github.com/OneBusAway/js-sdk/issues" }Verification
npm pack --dry-run→ no metadata warningsnpm install→ no peer-dep changesnpm run lint→ cleannpm test→ 12/12 passing (no test files added; metadata-only change)CLA
CLA has been signed via the cla-assistant.io form (the prior
@CLAassistantrecheck request was unable to confirm because theGitHub App requires a re-authenticated user session — manual sign
performed by codeboost-tr). Author email is also amended to the
GitHub-verified address; HEAD is
8b5e808.@CLAassistant — please re-run the check after this PR body update.