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

take screenshot for syntax error autoreload #439

Merged
merged 6 commits into from
Nov 13, 2024
Merged

take screenshot for syntax error autoreload #439

merged 6 commits into from
Nov 13, 2024

Conversation

dmail
Copy link
Member

@dmail dmail commented Nov 12, 2024

No description provided.

Copy link

github-actions bot commented Nov 12, 2024

File size impact

Impact on file sizes when merging small_fixes into main.

dist (+0.01%)
Files new size new gzip size
dist/jsenv_core.js 702 kB (+120 B / +0.02%) 156 kB (+43 B / +0.03%) ↗️
Unmodified (115) 449 kB (0 B / +0%) 123 kB (0 B / +0%) 👻
Total (116) 1.2 MB (+120 B / +0.01%) 279 kB (+43 B / +0.02%) ↗️
Generated by @jsenv/file-size-impact during file size impact#11800678138 on e09f926

Copy link

github-actions bot commented Nov 12, 2024

Performance impact

Impact on 38 metrics when merging small_fixes into main. Before drawing conclusion, keep in mind performance variability.

package metrics (-0.1%)
Metric Before merge After merge Impact
import duration 0.09 second 0.09 second -0.001 second / -1% ↘️
import memory heap used 6.3 MB 6.3 MB +3.3 kB / +0.05% ↗️
import memory heap total 15.2 MB 15.2 MB 👻
import fs open operations 75 75 👻
import fs stat operations 145 145 👻
npm tarball size 342 kB 342 kB +93 B / +0.03% ↗️
npm tarball unpacked size 1.6 MB 1.6 MB +243 B / +0.02% ↗️
npm tarball file count 206 206 👻
dev server metrics (+1%)
Metric Before merge After merge Impact
start duration 0.1 second 0.1 second +0.002 second / +2% ↗️
start memory heap used 6.7 MB 6.7 MB +3.8 kB / +0.06% ↗️
start memory heap total 24.2 MB 23.6 MB -524 kB / -2% ↘️
start fs read 0 0 👻
start fs write 0 0 👻
time to app display 0.5 second 0.5 second -0 second / -0.1% ↘️
serve app memory heap total 18.7 MB 19 MB +262 kB / +1% ↗️
serve app memory heap used 16.6 MB 16.6 MB -4.8 kB / -0.03% ↘️
serve app fs read 0 0 👻
serve app fs write 888 888 👻
time to 2nd app display 0.2 second 0.2 second +0.003 second / +1% ↗️
2nd serve memory heap total 2.6 MB 2.9 MB +262 kB / +10% ↗️
2nd serve memory heap used 706 kB 715 kB +8.7 kB / +1% ↗️
2nd serve fs read 0 0 👻
2nd serve fs write 888 888 👻
build metrics (+0.4%)
Metric Before merge After merge Impact
build duration 0.8 second 0.8 second +0.009 second / +1% ↗️
build memory heap used 32.2 MB 32.2 MB +9.7 kB / +0.03% ↗️
build memory heap total 65.7 MB 66 MB +262 kB / +0.4% ↗️
number of fs read operation 0 0 👻
number of fs write operation 264 264 👻
test metrics (-0.6%)
Metric Before merge After merge Impact
test plan duration 2.9 seconds 2.9 seconds -0.03 second / -1% ↘️
test plan memory heap used 43 MB 43.1 MB +17.5 kB / +0.04% ↗️
test plan memory heap total 88.9 MB 87.5 MB -1.3 MB / -1% ↘️
number of fs read operation 0 0 👻
number of fs write operation 1656 1656 👻
test metrics (serie) (+0.3%)
Metric Before merge After merge Impact
test plan duration 3.5 seconds 3.6 seconds +0.05 second / +1% ↗️
test plan memory heap used 43.1 MB 43.1 MB +7.1 kB / +0.02% ↗️
test plan memory heap total 87.5 MB 87.3 MB -262 kB / -0.3% ↘️
number of fs read operation 0 0 👻
number of fs write operation 1656 1656 👻
Generated by @jsenv/performance-impact during performance_impact#11800678125 on e09f926

@dmail dmail marked this pull request as ready for review November 12, 2024 15:31
@dmail dmail merged commit c9eb8b0 into main Nov 13, 2024
4 of 8 checks passed
@dmail dmail deleted the small_fixes branch November 13, 2024 09:45
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.

1 participant