Closed SethFalco closed 8 months ago
Merging #188 (463013d) into main (4828679) will increase coverage by
0.04%
. The diff coverage is0.00%
.
@@ Coverage Diff @@
## main #188 +/- ##
==========================================
+ Coverage 54.01% 54.06% +0.04%
==========================================
Files 11 11
Lines 1083 1082 -1
Branches 45 45
==========================================
Hits 585 585
+ Misses 498 497 -1
Files | Coverage ฮ | |
---|---|---|
src/ipx.ts | 68.54% <0.00%> (+0.27%) |
:arrow_up: |
๐ Linked issue
โ Type of change
๐ Description
I just released SVGO v3.0.3, which includes the revamped
removeScriptElement
plugin.This should do exactly what
removeXSS
was doing before in this repository, so that can be dropped in favor of the built-in plugin. In the next major version, this plugin will be renamed toremoveScripts
. I'll be happy to open another PR here when it can be referred to by its new name, along with any other steps to migrate between versions if required.You're welcome to review the implementation of it here under
/svgo/plugins/removeScriptElement.js
:If you have any problems, feel free to report an issue or hit me up. In theory, this PR should have identical behavior, there's just no need to package the
svgo-xss.ts
file.๐ Checklist