Do i need to just delete the patch file to get prep-deps passing? also there are lavamoat policies for node-sass that should be removed but wanted to get a thumbs up / greenlight before doing that @kumavis / @EtDu

@brad-decker Instructions on how to manage lavamoat policies and allow-scripts configuration are here: https://github.com/MetaMask/metamask-extension#changing-dependencies

And yes, deleting the patch would be the correct approach. That reminds me - we don't document how to deal with patches yet 🤔 I should address that.

Erg... the policy file changes made me look at bundles

https://bundlephobia.com/result?p=gulp-dart-sass@1.0.2 vs https://bundlephobia.com/result?p=gulp-sass@4.1.0

2.2mb minified vs 141kb minified... I'm thinking we just keep node-sass around for a while.

edit: Well, it's mostly sass which we already include. shrug

That is misleading because of node-sass, which downloads a binary as part of the install script I think? Thus hiding the true size.

Plus the bulk of the size of the dart package is due to the sass package, which we already have in our dependency tree. I do think this would be a strict reduction in our dependencies.

I have just rebased this to fix the yarn.lock conflict. It resolved itself automatically after rebasing and running yarn setup.