-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fix vulnerabilities and update packages. #21
Conversation
184f763
to
6f97d9e
Compare
6f97d9e
to
4d19674
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Tats, Left some minor comments. Thank you for this PR.
FYI resource I found for tslint --> eslint conversion, https://github.com/typescript-eslint/tslint-to-eslint-config. I am not sure if this is the best resource. Just added here for reference. Thanks again.
Thank you so much ❤️❤️🙏☕️ @hsubramanianaks yep, hence if you see the PR it adds Also for rest I have made apt info et. al. , thanks. |
Taking advantage of holiday work and hence opening this PR as one for all PR and avoid 1-1 PRs opened by bots which could cause issues because of the way individually they update, especially these runs are based on
packages-lock.json
which also contains dependent packages which are not taken into consideration while bot scans the file.Following PR also take care of few packages which are are deprecated, or really old and no development has happened around them for last 4 years.
usecolors
under this PR : Remove deprecated methods mochajs/mocha#4178Rest of the changes are fairly dependent on the audit and errors. We do need proper testing for this, to record all the functionality working around this.
Thanks heaps! ❤️☕️🙏