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

v4 #326

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

v4 #326

wants to merge 4 commits into from

Conversation

lagunovsky
Copy link

@lagunovsky lagunovsky commented Oct 16, 2021

  • Update svgo to v2
  • Update babel to v7
  • Using jest instead of tap
  • Improved type coverage
  • Generate named components

#307 #295 #303 #323 #318

update svgo to v2
update babel to v7
update eslint and prettier
tape replaced by jest
more complete type coverage
@afdev82
Copy link

afdev82 commented Mar 15, 2022

What is the status of this PR?
It would be great to update the SVGO package to the latest v2.

Currently the version of SVGO used doesn't allow to fix the vulnerability of the nth-check package.
The current version (1.3.2) uses css-select v2 (latest is 2.1.0), that depends on nth-check v1 (latest is 1.0.2).
The SVGO v2.8.0 uses css-select v4, that depends on nth-check v2, that is fine.

Could I help in some way to get it done?

@damdeez
Copy link

damdeez commented Apr 27, 2022

What is the status of this PR? It would be great to update the SVGO package to the latest v2.

Currently the version of SVGO used doesn't allow to fix the vulnerability of the nth-check package. The current version (1.3.2) uses css-select v2 (latest is 2.1.0), that depends on nth-check v1 (latest is 1.0.2). The SVGO v2.8.0 uses css-select v4, that depends on nth-check v2, that is fine.

Could I help in some way to get it done?

+1

@Miindaugas
Copy link

Miindaugas commented Mar 3, 2023

+1 high severity vulnerabilities in svgo 1.3.2.

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.

4 participants