BREAKING CHANGE : upgrade nodejs support till 24 and fix security vulns#110
BREAKING CHANGE : upgrade nodejs support till 24 and fix security vulns#110samarpanB wants to merge 3 commits into
Conversation
|
@appurva21 can you please review this ? All the test cases are passing in this now. You can check that here - samarpanB#1 . |
|
Hello @samarpanB , There are many changes in this PR. Are all the changes related to this CVE? |
The problem is, in order to fix this CVE I need to update the nodejs version and hence its related dependencies also needed an upgrade. Without this, the CVE fix wasn't feasible. To ensure nothing is broken, I have ensured test cases are working. |
Here is a PR for that #111. But I am not sure if test cases are passing on this one. That was my issue and concern. Hence this PR got big. |
…lnerabilities Signed-off-by: Samarpan Bhattacharya <this.is.samy@gmail.com>
BREAKING CHANGE
This PR fixes a lot of security vulnerabilities which are showing up through npm audit. It also adds support for latest active and LTS nodejs versions. It removes non-maintained linter and coverage tools and replaces them with drop-in replacements. No logical or code change has been done except for lint issue fixes.
PR Checklist:
npm testlocally and all tests are passing.PR Description