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

feat(ci): add breaking ci notification #30283

Closed
wants to merge 17 commits into from
Closed

Conversation

desi
Copy link
Contributor

@desi desi commented Feb 12, 2025

Description

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

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 Feb 12, 2025
@desi desi force-pushed the dm-add-ci-break-notification branch from cb40d1e to 9bf23d3 Compare February 12, 2025 23:20
@desi desi changed the title WIP: add breaking ci notification feat(ci): add breaking ci notification Feb 12, 2025
@desi desi force-pushed the dm-add-ci-break-notification branch 9 times, most recently from 9fd6d11 to 154ce6b Compare February 18, 2025 14:54
@desi desi force-pushed the dm-add-ci-break-notification branch from 154ce6b to 0a75291 Compare February 18, 2025 15:21
@metamaskbot
Copy link
Collaborator

Builds ready [0a75291]
Page Load Metrics (1560 ± 57 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint14361876156711656
domContentLoaded14241856154311857
load14331884156011957
domInteractive168833188
backgroundConnect85526168
firstReactRender1597432914
getState4508105
initialActions01000
loadScripts9681411111011254
setupStore6481094
uiStartup16172206180417986
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@metamaskbot
Copy link
Collaborator

Builds ready [e30a279]
Page Load Metrics (1929 ± 105 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint38524801843401192
domContentLoaded151523991883209101
load153324061929219105
domInteractive24120472512
backgroundConnect8119463215
firstReactRender1587322210
getState7179364019
initialActions02000
loadScripts10421814138117886
setupStore96519188
uiStartup184029072231263126
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@metamaskbot
Copy link
Collaborator

Builds ready [080c3a8]
Page Load Metrics (1744 ± 76 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint14362128174615273
domContentLoaded14282106172315172
load14362135174415876
domInteractive218439189
backgroundConnect105927168
firstReactRender1591382613
getState563192010
initialActions01000
loadScripts10261507124511957
setupStore85819188
uiStartup166524962023214103
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@desi desi closed this Feb 20, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Feb 20, 2025
@desi desi deleted the dm-add-ci-break-notification branch February 20, 2025 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team-extension-platform Extension Platform team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants