fix(#220): skip <script> and <style> with SRI #221

SukkaW · 2023-01-08T17:26:11Z

The PR fixes #220.

The PR makes htmlnano skip any <script /> and <style /> tags that have SRI attributes, including minifyJs, minifyCss, minifyJson, mergeStyles and mergeScripts.
The reason behind the change is documented in the comment, and test cases about the change are also added.

maltsev

👍

SukkaW added 4 commits January 9, 2023 01:02

fix(posthtml#220): should not minify <script> and <style> with SRI

5478092

fix(posthtml#220): should not merge <style> or <script> with SRI

9681eaa

fix(posthtml#220): should not minify json with SRI

447079d

chore(posthtml#220): add comments, documenting why SRI should be skipped

e795aa2

SukkaW changed the title ~~fix(#220): should not minify <script> and <style> with SRI~~ fix(#220): skip <script> and <style> with SRI Jan 8, 2023

maltsev approved these changes Jan 8, 2023

View reviewed changes

maltsev merged commit 688cf37 into posthtml:master Jan 8, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(#220): skip <script> and <style> with SRI #221

fix(#220): skip <script> and <style> with SRI #221

SukkaW commented Jan 8, 2023

maltsev left a comment

fix(#220): skip <script> and <style> with SRI #221

fix(#220): skip <script> and <style> with SRI #221

Conversation

SukkaW commented Jan 8, 2023

maltsev left a comment

Choose a reason for hiding this comment