search

jsenv / importmap-node-module

Generate importmap for node_modules
35 stars 4 forks source link

Rename terminalImportMap into manualImportMap, remove initialImportMap #47

Closed dmail closed 3 years ago

dmail commented 3 years ago

2.6.0

github-actions[bot] commented 3 years ago

Performance impact

Impact on 9 metrics when merging manual_importmap into master. Before drawing conclusion, keep in mind performance variability.

package metrics (+0.35%)
Metric Before merge After merge Impact
import duration 0.29 seconds 0.295 seconds +0.005 seconds / +1.72% :arrow_upper_right:
import memory heap used 6.66 MB 6.65 MB -9.08 KB / -0.13% :arrow_lower_right:
npm tarball size 83.14 KB 83.2 KB +66 B / +0.08% :arrow_upper_right:
npm tarball unpacked size 367.65 KB 367.86 KB +214 B / +0.06% :arrow_upper_right:
npm tarball file count 26 26 :ghost:
write importmap metrics (-0.84%)
Metric Before merge After merge Impact
write importmap duration 0.318 seconds 0.31 seconds -0.008 seconds / -2.52% :arrow_lower_right:
write importmap memory heap used 6.72 MB 6.72 MB +392 B / +0.01% :arrow_upper_right:
number of fs read operation 0 0 :ghost:
number of fs write operation 8 8 :ghost:

Generated by @jsenv/performance-impact during performance_impact#1463020525 on 52d085a

codecov[bot] commented 3 years ago

Codecov Report

Merging #47 (445f9d0) into master (f62d473) will increase coverage by 0.19%. The diff coverage is 56.75%.

:exclamation: Current head 445f9d0 differs from pull request most recent head 6c4d4b5. Consider uploading reports for the commit 6c4d4b5 to get more accurate results