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

Have mmi build being generated #19441

Merged
merged 4 commits into from
Jun 6, 2023
Merged

Conversation

zone-live
Copy link
Contributor

@zone-live zone-live commented Jun 5, 2023

In the past the Lavamoat policy update script was changed to skip MMI build type, but now it can be updated to remove that and include it.
By running yarn lavamoat:auto we can now see that the build is running successfully and the policy file is generated for MMI.

Screenshot 2023-06-01 at 21 02 26

Pre-merge author checklist

  • I've clearly explained:
    • What problem this PR is solving
    • How this problem was solved
    • How reviewers can test my changes
  • Sufficient automated test coverage has been added

Pre-merge reviewer checklist

  • Manual testing (e.g. pull and build branch, run in browser, test code being changed)
  • PR is linked to the appropriate GitHub issue
  • IF this PR fixes a bug in the release milestone, add this PR to the release milestone

If further QA is required (e.g. new feature, complex testing steps, large refactor), add the Extension QA Board label.

In this case, a QA Engineer approval will be be required.

@zone-live zone-live requested review from a team and kumavis as code owners June 5, 2023 09:22
@zone-live zone-live requested a review from chloeYue June 5, 2023 09:22
@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2023

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.

@zone-live zone-live requested a review from Gudahtt June 5, 2023 09:25
@metamaskbot
Copy link
Collaborator

Builds ready [cdd03dd]
Page Load Metrics (1604 ± 53 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint96166116168
domContentLoaded13861904157611153
load14791904160411153
domInteractive13861904157611153
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@codecov

This comment was marked as outdated.

@metamaskbot
Copy link
Collaborator

Builds ready [26b36ab]
Page Load Metrics (1682 ± 77 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1001771282311
domContentLoaded14682058167816378
load14872058168216177
domInteractive14682058167816378
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [7f8bfa6]
Page Load Metrics (1833 ± 52 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint981931342612
domContentLoaded15672072178211857
load16432073183310752
domInteractive15672072178211857
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [2eca58c]
Page Load Metrics (1555 ± 37 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint98153121178
domContentLoaded1355172115487938
load1374172115557837
domInteractive1355172115487938
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@zone-live zone-live merged commit 51f6a29 into develop Jun 6, 2023
@zone-live zone-live deleted the MMI-3231-create-MMI-build-from-MM branch June 6, 2023 15:43
@github-actions github-actions bot locked and limited conversation to collaborators Jun 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants