Skip to content

fix: Disable MV3 for MV2 Flask production build#25209

Merged
FrederikBolding merged 2 commits intodevelopfrom
fb/disable-mv3-for-flask-mv2-prod
Jun 11, 2024
Merged

fix: Disable MV3 for MV2 Flask production build#25209
FrederikBolding merged 2 commits intodevelopfrom
fb/disable-mv3-for-flask-mv2-prod

Conversation

@FrederikBolding
Copy link
Copy Markdown
Member

@FrederikBolding FrederikBolding commented Jun 11, 2024

Description

Disables MV3 for the MV2 Flask production build. This was seemingly missed when setting up the CI to build for production and caused problems with the latest build submitted to Firefox.

Also updates the release script to use MV2 builds for Firefox.

Open in GitHub Codespaces

@FrederikBolding FrederikBolding added the team-extension-platform Extension Platform team label Jun 11, 2024
@FrederikBolding FrederikBolding requested review from a team and kumavis as code owners June 11, 2024 10:16
@metamaskbot metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Jun 11, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.59%. Comparing base (1e18d45) to head (2c82e65).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #25209   +/-   ##
========================================
  Coverage    65.59%   65.59%           
========================================
  Files         1362     1362           
  Lines        54167    54167           
  Branches     14167    14167           
========================================
  Hits         35527    35527           
  Misses       18640    18640           

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

@FrederikBolding FrederikBolding merged commit bc6acbe into develop Jun 11, 2024
@FrederikBolding FrederikBolding deleted the fb/disable-mv3-for-flask-mv2-prod branch June 11, 2024 11:45
@github-actions github-actions bot locked and limited conversation to collaborators Jun 11, 2024
@metamaskbot metamaskbot added the release-12.1.0 Issue or pull request that will be included in release 12.1.0 label Jun 11, 2024
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [2c82e65]
Page Load Metrics (133 ± 180 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint66897873
domContentLoaded8131011
load401764133374180
domInteractive8131011
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@metamaskbot metamaskbot added release-11.16.10 Issue or pull request that will be included in release 11.16.10 and removed release-12.1.0 Issue or pull request that will be included in release 12.1.0 labels Jun 12, 2024
@metamaskbot
Copy link
Copy Markdown
Collaborator

Missing release label release-11.16.10 on PR. Adding release label release-11.16.10 on PR and removing other release labels(release-12.1.0), as PR was cherry-picked in branch 11.16.10.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template release-11.16.10 Issue or pull request that will be included in release 11.16.10 team-extension-platform Extension Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants