Skip to content
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

Merged
merged 2 commits into from Dec 26, 2022

Conversation

Tatsinnit
Copy link
Member

@Tatsinnit Tatsinnit commented Dec 18, 2022

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.

Rest 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! ❤️☕️🙏

Copy link
Collaborator

@hsubramanianaks hsubramanianaks left a 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.

@Tatsinnit
Copy link
Member Author

Tatsinnit commented Dec 20, 2022

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 eslint

Screenshot 2022-12-20 at 5 05 08 PM

Also for rest I have made apt info et. al. , thanks.

@Tatsinnit Tatsinnit self-assigned this Dec 20, 2022
@Tatsinnit Tatsinnit merged commit 2c7dc0a into Azure:main Dec 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🧑‍🔧💡 New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants