Skip to content

Conversation

@HowardBraham
Copy link
Contributor

@HowardBraham HowardBraham commented Jul 28, 2025

Description

This PR is an excellent example of MetaMask/MetaMask-planning#2676 where we should be skipping most CI steps.

Changelog

CHANGELOG entry: null

Related issues

Fixes problem in: #34530
Example of: MetaMask/MetaMask-planning#2676

@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-extension-platform Extension Platform team label Jul 28, 2025
@HowardBraham HowardBraham added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Jul 28, 2025
@HowardBraham HowardBraham self-assigned this Jul 28, 2025
@HowardBraham HowardBraham marked this pull request as ready for review July 28, 2025 16:42
@HowardBraham HowardBraham enabled auto-merge July 28, 2025 16:42
@metamaskbot
Copy link
Collaborator

Builds ready [61ee57a]
UI Startup Metrics (1309 ± 67 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1309117616166713441434
load1132100414576711721228
domContentLoaded112498014456911631223
domInteractive18135371735
firstPaint69079145145911361201
backgroundConnect25524034112258274
firstReactRender2215159152134
getState9310711925
initialActions3012213113
loadScripts876733118869915974
setupStore74334716
WebpackHomeuiStartup23541823296221224292785
load18661364240922419822304
domContentLoaded18491357228820619742160
domInteractive2113205301461
firstPaint1586343567173302
backgroundConnect61113118940301
firstReactRender1478833062185292
getState2842766215229
initialActions8223823812
loadScripts18441353227520519702149
setupStore185263371530
FirefoxBrowserifyHomeuiStartup14541275205012914881705
load1255110916158712861450
domContentLoaded1255110916158712861450
domInteractive1103646571107299
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect251466102651
firstReactRender282063112959
getState15219734670
initialActions507510226
loadScripts1233109315948612631429
setupStore8314315622
WebpackHomeuiStartup18561599239219819672272
load15731344202716516731920
domContentLoaded15731344202716516731920
domInteractive1133740264115310
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect3321240233747
firstReactRender564310396168
getState10381101224
initialActions9120923629
loadScripts15481318200616316491892
setupStore19525046971
Benchmark value 1310 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1133 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1124 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 256 exceeds gate value 10 for chrome browserify home mean backgroundConnect
Benchmark value 4 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 877 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1435 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1229 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1223 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 1201 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 275 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 13 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 974 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2354 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1867 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1849 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 22 exceeds gate value 21 for chrome webpack home mean domInteractive
Benchmark value 61 exceeds gate value 40 for chrome webpack home mean backgroundConnect
Benchmark value 8 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 1844 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2785 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2305 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2161 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 61 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 302 exceeds gate value 90 for chrome webpack home p95 backgroundConnect
Benchmark value 229 exceeds gate value 195 for chrome webpack home p95 getState
Benchmark value 12 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2150 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1455 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1255 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1255 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 29 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 15 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 5 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 1233 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 1705 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 299 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 59 exceeds gate value 55 for firefox browserify home p95 firstReactRender
Benchmark value 70 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 26 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 1856 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1574 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1574 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 114 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 33 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 56 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 9 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1548 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 19 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2272 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1920 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1920 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 310 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 68 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 29 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1892 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 71 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 2090ms | Sum of p95 exceeds: 3256.8ms
Sum of all benchmark exceeds: 5346.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 43 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 15 Bytes (0%)

Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@davidmurdoch davidmurdoch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's probably fine :-)

@HowardBraham HowardBraham added this pull request to the merge queue Jul 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 29, 2025
@HowardBraham HowardBraham added this pull request to the merge queue Jul 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 30, 2025
@HowardBraham HowardBraham added this pull request to the merge queue Jul 30, 2025
Merged via the queue into main with commit ed4493d Jul 30, 2025
155 of 156 checks passed
@HowardBraham HowardBraham deleted the newline branch July 30, 2025 18:38
@github-actions github-actions bot locked and limited conversation to collaborators Jul 30, 2025
@metamaskbot metamaskbot added the release-13.1.0 Issue or pull request that will be included in release 13.1.0 label Jul 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-13.1.0 Issue or pull request that will be included in release 13.1.0 team-extension-platform Extension Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants