Rich-Harris / butternut

The fast, future-friendly minifier
https://butternut.now.sh
MIT License
1.17k stars 17 forks source link

[WIP] Skip nodes by default #63

Closed Rich-Harris closed 7 years ago

Rich-Harris commented 7 years ago

This switches things around a bit so that nodes are skipped by default, and only unskipped if they are initialised (and are not, say, an EmptyStatement). This will allow us to do less work and be a little bit smarter about certain things.

codecov[bot] commented 7 years ago

Codecov Report

Merging #63 into master will decrease coverage by 0.4%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage    88.2%   87.79%   -0.41%     
==========================================
  Files          66       67       +1     
  Lines        1424     1409      -15     
==========================================
- Hits         1256     1237      -19     
- Misses        168      172       +4
Impacted Files Coverage Δ
src/program/types/index.js 100% <ø> (ø) :arrow_up:
src/program/types/VariableDeclarator.js 100% <ø> (ø) :arrow_up:
src/program/Scope.js 93.97% <ø> (-0.15%) :arrow_down:
src/program/wrap.js 100% <100%> (ø) :arrow_up:
src/program/types/FunctionDeclaration.js 100% <100%> (ø) :arrow_up:
src/program/types/IfStatement.js 95.21% <100%> (+0.02%) :arrow_up:
src/program/types/VariableDeclaration.js 100% <100%> (ø) :arrow_up:
src/program/types/EmptyStatement.js 100% <100%> (ø)
src/program/types/ClassDeclaration.js 66.66% <100%> (-33.34%) :arrow_down:
src/program/Node.js 87.03% <100%> (+0.24%) :arrow_up:
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2dcde0b...c697b71. Read the comment docs.