Skip to content

Nit fixes for story-test integration#17344

Merged
NidhiKJha merged 3 commits intodevelopfrom
test-storybook-followup
Jan 25, 2023
Merged

Nit fixes for story-test integration#17344
NidhiKJha merged 3 commits intodevelopfrom
test-storybook-followup

Conversation

@NidhiKJha
Copy link
Copy Markdown
Member

This is a follow-up PR for story-test integration PR.

Updated the PR with the fixes @Gudahtt suggested.

Changes Made:

  1. moved playwright from dependencies to dev dependencies.
  2. Removed the lint comment from ui/components/app/app-header/app-header.stories.js
  3. Updated comment in storybook.js

@NidhiKJha NidhiKJha requested review from a team and kumavis as code owners January 23, 2023 13:49
@NidhiKJha NidhiKJha requested review from Gudahtt and montelaidev and removed request for montelaidev January 23, 2023 13:49
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [cf1e9fc]
Page Load Metrics (1317 ± 130 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1032211342612
domContentLoaded105120831303261126
load105120831317272130
domInteractive105020831303261126
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@NidhiKJha NidhiKJha force-pushed the test-storybook-followup branch from cf1e9fc to d1a62b5 Compare January 23, 2023 17:10
@NidhiKJha NidhiKJha requested a review from Gudahtt January 23, 2023 17:10
Copy link
Copy Markdown
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, thanks!

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [d1a62b5]
Page Load Metrics (1315 ± 107 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint901711212010
domContentLoaded104217041292223107
load106117041315223107
domInteractive104217031292223107
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [1b65e29]
Page Load Metrics (1335 ± 141 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint103182126189
domContentLoaded101418421316294141
load101519091335294141
domInteractive101418421316294141
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #17344 (1b65e29) into develop (cf10b2a) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #17344      +/-   ##
===========================================
- Coverage    60.07%   60.06%   -0.01%     
===========================================
  Files          936      936              
  Lines        36049    36049              
  Branches      9261     9261              
===========================================
- Hits         21654    21651       -3     
- Misses       14395    14398       +3     
Impacted Files Coverage Δ
app/scripts/metamask-controller.js 60.49% <0.00%> (-0.25%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@NidhiKJha NidhiKJha merged commit 8a6bf4d into develop Jan 25, 2023
@NidhiKJha NidhiKJha deleted the test-storybook-followup branch January 25, 2023 03:54
@github-actions github-actions bot locked and limited conversation to collaborators Jan 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants