-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat: upgradeDependency
utility
#82
Conversation
Codecov Report
@@ Coverage Diff @@
## main #82 +/- ##
===========================================
- Coverage 98.02% 83.33% -14.70%
===========================================
Files 5 5
Lines 405 492 +87
Branches 64 64
===========================================
+ Hits 397 410 +13
- Misses 8 82 +74
|
Not sure how I missed adding those deps 😅 Thanks for taking the time on fixing these... |
Thanks for PR dear @rishi-raj-jain. I have made few more refactors (it is halfway done sorry in between projects). I think this is a really good start. Will followup later. |
Awesome! Do let me know if I can contribute. I think nypm shall also have like each function to do things, addDep, checkDep, updateDep, etc. But yeah ik the time is limited |
Hi and thanks for this PR again. I think it might be better to rethink about this functionality after introducing |
fix #78