Rename jshintrc to .jshintrc#421
Conversation
|
#134 is where the |
|
I'm split. On one hand, that is the proper name of the file for a project. On the other, we aren't actually linting any code in our linters directory, and users are expected to copy the file into their project anyways. Dotfiles are harder to work with: by default my editor hides them in the sidebar, and I can't tab complete them on the terminal. I would prefer to keep this as |
|
@justjake I didn't realize that this was intentional. Seeing as how the other file in the directory is a dotfile, and the pull request I referenced says that it adds a |
|
it was probably a mistake back then -- i was expressing my personal opinion |
|
I'd love to hear some more opinions on this - personally, I don't find dotfiles difficult to work with, and I already have to configure my editor to show dotfiles to work with |
02c8853 to
2520732
Compare
|
+1 it's better to copy over the file without having to rename it. Plus someone could forget to rename it and then it won't work for them. |
|
I think it should be changed to a dotfile because:
|
|
@chrisngobanh that's a strong argument too. (would you mind freshly rebasing this on latest master, btw?) |
According to http://jshint.com/docs/, the "special file" to configure JSHint is named `.jshintrc`, not `jshintrc`.
2520732 to
8989a8a
Compare
|
@ljharb Done! Side question: Is the |
|
Somehow I doubt it. We may end up removing the file anyways - jshint's ES6 support is limited. |
According to http://jshint.com/docs/, the "special file" to configure JSHint is named
.jshintrc, notjshintrc.