fix: remove unused native bcrypt package (issue #40)#2588
fix: remove unused native bcrypt package (issue #40)#2588tmdeveloper007 wants to merge 1 commit into
Conversation
|
Someone is attempting to deploy a commit to the Nisshchaya's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThe change updates package.json to remove the bcrypt dependency and add bcryptjs, consolidating password hashing to a single library. ChangesDependency migration
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested labels: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
CodeRabbit review is passing. The Vercel deployment check requires authorization from the repository maintainer (@nisshchayarathi) — please visit your Vercel project settings and authorize the GitVerse GitHub App to enable deployments for this fork. Happy to help if needed. |
Summary
Removes the unused
bcryptnative C++ package frompackage.jsondependencies. The project exclusively usesbcryptjsthroughout the codebase.Issue
Fixes #40: Fix duplicate bcrypt dependencies (bcrypt + bcryptjs)
Changes
"bcrypt": "^6.0.0"fromdependenciesinpackage.jsonbcryptjs(no import changes required)Testing
npm run typecheckpassesbcryptpackageFiles Changed
package.jsonSummary by CodeRabbit