Snider

Results 26 comments of Snider

@coderabbitai the code walkthrough has an incorect statement. ``` Narrowed generated-files pattern from node_modules/** to node_modules/.bin/ for the install:frontend:deps task, reducing the scope of generated file tracking. ``` should be...

> 🧩 Analysis chain > @Snider: You're absolutely correct! Thank you for catching that error. > > Looking at the actual diff, the change was indeed from `node_modules/*` to `node_modules/.bin/*`,...

> Did you mean to review the comments? We need to leave them in so new contributors know where to add things. Oh, that's just me being a noob trying...

> Why do we only want to check the bin directory for module changes? Because using the level above makes Task checksum the entire node_modules folder to check for changes,...

> Actually this should probably just be package.json and forget the rest 😅 Yeah, someone who uses Task more can probably improve the change; so that "npm install" and "task...

I would argue the App is not ok as you reported, 3.5GB is interesting in a bad way. Wails is just rendering a WebView, it's your app that is crashing,...