LearnBoost / cluster

Node.JS multi-core server manager with plugins support.
http://learnboost.github.com/cluster
MIT License
2.29k stars 159 forks source link

Eliminating some false positives on the Reload plugin #68

Closed solsys closed 13 years ago

solsys commented 13 years ago

Vim likes to throw .myfile.js.swp files in the same directory as myfile.js. This has the nasty side-effect of forcing Cluster to restart every few keystrokes. This patch just changes the reload filter to only trigger on files with an extension of .js, instead of files that happen to have .js somewhere in the name.

eirikurn commented 13 years ago

This line is changed in my branch to support other file extensions which in the progress fixes this issue. I'll be creating another pull request later today.

tj commented 13 years ago

I'm going to close this, I will fix this extension issue but path.extname() is a bit cleaner