Skip to content

refactor: make manifest content scripts relative to their dev location#23672

Merged
davidmurdoch merged 1 commit intodevelopfrom
manifest-relative-paths
Mar 27, 2024
Merged

refactor: make manifest content scripts relative to their dev location#23672
davidmurdoch merged 1 commit intodevelopfrom
manifest-relative-paths

Conversation

@davidmurdoch
Copy link
Copy Markdown
Contributor

@davidmurdoch davidmurdoch commented Mar 23, 2024

The webpack build system I'm working on uses the extension manifest JSON files to determine the entry points of the application. The manifest file sometimes references files using a path relative to both the repo and the bundled extension (background.html and popup.html) and sometimes it doesn't (inpage.js lives in the scripts folder).

This PR makes the paths relative to the repo (and the bundled extension).

@davidmurdoch davidmurdoch added the team-extension-platform Extension Platform team label Mar 23, 2024
@github-actions
Copy link
Copy Markdown
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.

@davidmurdoch davidmurdoch force-pushed the manifest-relative-paths branch 2 times, most recently from 5eda770 to d93059c Compare March 23, 2024 18:01
@davidmurdoch davidmurdoch force-pushed the manifest-relative-paths branch from d93059c to 2c72e10 Compare March 23, 2024 20:25
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [2c72e10]
Page Load Metrics (1046 ± 504 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint822061293215
domContentLoaded147333147
load69255310461050504
domInteractive147333147
Bundle size diffs [🚀 Bundle size reduced!]
  • background: -935.51 KiB (-25.33%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@davidmurdoch davidmurdoch marked this pull request as ready for review March 23, 2024 22:39
@davidmurdoch davidmurdoch requested review from a team and kumavis as code owners March 23, 2024 22:39
@davidmurdoch davidmurdoch merged commit 207f6af into develop Mar 27, 2024
@davidmurdoch davidmurdoch deleted the manifest-relative-paths branch March 27, 2024 22:03
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2024
@metamaskbot metamaskbot added the release-11.15.0 Issue or pull request that will be included in release 11.15.0 label Mar 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants