ci: tighten size threshold to 1% or 500 bytes (#33969)
The size diff threshold of 1% has proven to be too lenient for us to catch size regressions in AIO. Since the AIO main bundle is between 400-500 KB, a size regression must be between 4-5 KB before it will cause the tests to fail. As a result, we may merge many changes with smaller regressions of a few KB before the size test eventually lets us know that the number has increased. The hope is that lowering the threshold will help us catch the smaller regressions during code review and prevent the size tests failing at a random later time when someone catches the size "hot potato". PR Close #33969
This commit is contained in:

committed by
Matias Niemelä

parent
ff1be4cd28
commit
d752e26eb2
@ -3,7 +3,7 @@
|
||||
"master": {
|
||||
"uncompressed": {
|
||||
"runtime-es2015": 1485,
|
||||
"main-es2015": 141575,
|
||||
"main-es2015": 142186,
|
||||
"polyfills-es2015": 36808
|
||||
}
|
||||
}
|
||||
@ -21,7 +21,7 @@
|
||||
"master": {
|
||||
"uncompressed": {
|
||||
"runtime-es2015": 1485,
|
||||
"main-es2015": 147719,
|
||||
"main-es2015": 148320,
|
||||
"polyfills-es2015": 36808
|
||||
}
|
||||
}
|
||||
@ -49,7 +49,7 @@
|
||||
"master": {
|
||||
"uncompressed": {
|
||||
"runtime-es2015": 2289,
|
||||
"main-es2015": 227998,
|
||||
"main-es2015": 227232,
|
||||
"polyfills-es2015": 36808,
|
||||
"5-es2015": 779
|
||||
}
|
||||
@ -60,7 +60,7 @@
|
||||
"uncompressed": {
|
||||
"bundle": "TODO(i): temporarily increase the payload size limit from 105779 - this is due to a closure issue related to ESM reexports that still needs to be investigated",
|
||||
"bundle": "TODO(i): we should define ngDevMode to false in Closure, but --define only works in the global scope.",
|
||||
"bundle": 177447
|
||||
"bundle": 176433
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user