neumino
commented
11 years ago Yes, I totally agree!
The current config.js should be rename config.sample.js and we should have config.js in the gitignore
Closed eiriksm closed 11 years ago
neumino
commented
11 years ago Yes, I totally agree!
The current config.js should be rename config.sample.js and we should have config.js in the gitignore
Just a suggestion, but I personally prefer to have this kind of file in .gitignore
Instead we could have a config-template, and add an extra step in the installation guide.
This way I don't have to see my config changes as diff in git, and - more important - I don't accidentally commit some kind of sensitive data (or override with some crazy new defaults).
Let me know what you think, and I'll submit a pull request for it if you want!
And again: Awesome work!