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 dev dependencies vulnerabilities #236

Merged

Conversation

fbosquet-azpn
Copy link
Contributor

@fbosquet-azpn fbosquet-azpn commented Mar 13, 2024

Fixing some vulnerabilities in dev dependencies

Before

npm install

up to date, audited 1111 packages in 3s

108 packages are looking for funding
  run `npm fund` for details

50 vulnerabilities (44 moderate, 5 high, 1 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

After

npm install

added 52 packages, removed 201 packages, changed 228 packages, and audited 962 packages in 38s

125 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

npm run build, npm run test, npm run docs and npm run lint are still passing.

@fbosquet-azpn
Copy link
Contributor Author

@GuillaumeVix @johannesleite
Thanks for going to all the previous merge requests that I have marked as obsolete because of my previous work on storybook.

I have done the same thing, I have reviewed the remaining 28 active Pull request related to vulnerability and package upgrade.
I have added a comment on all the one that can be closed (25 out of 28).

@johannesleite johannesleite merged commit 86cdd7a into equinor:dev Mar 14, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants