framesurge / perseus

A state-driven web development framework for Rust with full support for server-side rendering and static generation.
https://framesurge.sh/perseus/en-US
MIT License
2.15k stars 89 forks source link

feat(perseus-cli): add 'node_modules' to watch exclusions #313

Closed gladion136 closed 6 months ago

gladion136 commented 10 months ago

The Problem

Like described inside this issue: 309 perseus serve -w is very slow when listening to a large number of files.

This change

This PR will not fix this in every case but it adds the node_modules folder to the watch exclusions. The 'node_modules' folder is similar to the target folder but from npm and it's often huge. I think a lot of developer has a node_modules folder because they are using bootstrap, tailwind or some other npm dependencies inside there perseus projects.

arctic-hen7 commented 9 months ago

Great idea, this will definitely streamline things significantly for a lot of people! Thanks, and sorry for the late reply!

arctic-hen7 commented 9 months ago

There seem to be some issues in perseus-rocket that are unrelated to this PR but causing the tests to fail, I'll investigate this...

arctic-hen7 commented 9 months ago

All fixed on main, syncing from there should work.

gladion136 commented 7 months ago

Sorry for the late reply! The branch is now rebased to main.