Skip to content

build: remove static copy of react-responsive-carousel styles as it is no longer used statically#23278

Merged
davidmurdoch merged 4 commits intodevelopfrom
remove-carousel-from-output
Mar 7, 2024
Merged

build: remove static copy of react-responsive-carousel styles as it is no longer used statically#23278
davidmurdoch merged 4 commits intodevelopfrom
remove-carousel-from-output

Conversation

@davidmurdoch
Copy link
Copy Markdown
Contributor

@davidmurdoch davidmurdoch commented Mar 1, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 1, 2024

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.

@davidmurdoch davidmurdoch added the team-extension-platform Extension Platform team label Mar 1, 2024
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [ffe9c11]
Page Load Metrics (1360 ± 357 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint862791475426
domContentLoaded1094422612
load7221011360743357
domInteractive1093422612
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@davidmurdoch davidmurdoch marked this pull request as ready for review March 1, 2024 22:27
@davidmurdoch davidmurdoch requested review from a team and kumavis as code owners March 1, 2024 22:27
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [a9bc7b9]
Page Load Metrics (1190 ± 399 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint742311193617
domContentLoaded105826147
load6122611190830399
domInteractive105826147
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.72%. Comparing base (be87264) to head (afc4f2c).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #23278   +/-   ##
========================================
  Coverage    68.72%   68.72%           
========================================
  Files         1105     1105           
  Lines        43250    43250           
  Branches     11562    11562           
========================================
  Hits         29720    29720           
  Misses       13530    13530           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [380aa09]
Page Load Metrics (1278 ± 377 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint762661324321
domContentLoaded98130199
load5820821278785377
domInteractive98130199
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@legobeat legobeat requested a review from a team March 7, 2024 10:31
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [afc4f2c]
Page Load Metrics (1212 ± 433 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint912161463215
domContentLoaded14104503014
load7722691212902433
domInteractive14104503014
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@davidmurdoch davidmurdoch merged commit a43c645 into develop Mar 7, 2024
@davidmurdoch davidmurdoch deleted the remove-carousel-from-output branch March 7, 2024 16:08
@github-actions github-actions bot locked and limited conversation to collaborators Mar 7, 2024
@metamaskbot metamaskbot added the release-11.14.0 Issue or pull request that will be included in release 11.14.0 label Mar 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-11.14.0 Issue or pull request that will be included in release 11.14.0 team-extension-platform Extension Platform team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Remove extraneous react carousel from build process

4 participants