Skip to content

Fix start:dev script#10399

Merged
Gudahtt merged 1 commit intodevelopfrom
fix-start-dev-script
Feb 8, 2021
Merged

Fix start:dev script#10399
Gudahtt merged 1 commit intodevelopfrom
fix-start-dev-script

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Feb 8, 2021

The start:dev script relies upon electron and sqlite, both of which had their install scripts disabled by allow-scripts. Without
those two packages, start:dev crashes immediately.

The manifest has been updated to ensure install scripts run for both packages. start:dev now works.

Manual testing steps:

  • Run yarn start:dev

The `start:dev` script relies upon `electron` and `sqlite`, both of
which had their install scripts disabled by `allow-scripts`. Without
those two packages, `start:dev` crashes immediately.

The manifest has been updated to ensure install scripts run for both
packages. `start:dev` now works.
@Gudahtt Gudahtt requested a review from a team as a code owner February 8, 2021 14:42
@Gudahtt Gudahtt requested a review from brad-decker February 8, 2021 14:42
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [5ca873f]
Page Load Metrics (547 ± 33 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint43715694
domContentLoaded3466235467033
load3486245477033
domInteractive3466235467033

@Gudahtt Gudahtt merged commit aadb8ac into develop Feb 8, 2021
@Gudahtt Gudahtt deleted the fix-start-dev-script branch February 8, 2021 16:06
@github-actions github-actions bot locked and limited conversation to collaborators Feb 8, 2021
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.

3 participants