Skip to content
This repository has been archived by the owner on Jun 12, 2023. It is now read-only.

Building snap locally takes over 3 minutes #251

Open
meandavejustice opened this issue Mar 20, 2023 · 1 comment
Open

Building snap locally takes over 3 minutes #251

meandavejustice opened this issue Mar 20, 2023 · 1 comment

Comments

@meandavejustice
Copy link
Collaborator

It appears that we don't have a watch command setup for the snap when running yarn demo, we should speed up the snap build command before we add watch so that it can be useful for local dev.

nard@wintermute:/mnt/c/Users/davej/Code/filsnap/packages/snap$ npm run build

> @chainsafe/[email protected] prebuild
> rm -rf build && rm -rf dist


> @chainsafe/[email protected] build
> yarn run build:snap
Build success: 'src/index.ts' bundled as 'dist/bundle.js'!
Manifest Success: Validated snap.manifest.json!
WARNING: Missing strong random number source
Eval Success: evaluated 'dist/bundle.js' in SES!

real    3m1.599s
user    0m20.466s
sys     1m4.116s
@mpetrunic
Copy link
Member

Pretty sure ses eval takes the longest, it could be disabled but there is a risk, Metamask won't install snap.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants