Added space after comment rule

[AdriVanHoudt]

Another eslint, another PR (until v1 :P)

• updated eslint to 0.23
• added space after comment rule, except when followed by , to allow comments for optional function params like function (title /*, options, fn */)

[AdriVanHoudt]

hmm tests fail because

lib/reporters/console.js:
        Line 43: space-unary-ops - Unary word operator "this" must be followed by whitespace.

And actually doing this .count++; does not fail any tests, weird

[AdriVanHoudt]

I talked to the people of eslint (see linked issue) and a fix will probably land this weekend so I would hold off on this PR until then, either we jump 2 releases or an extra patch, both are nice

[AdriVanHoudt]

@geek how do you want me to proceed? update the pr with #387 and just add the new rule, make a new pr with the rule or just close if you don't want the rule

[geek]

I would update the PR with the new rule... delete the change to package.json and it should merge. Right?

[AdriVanHoudt]

My rebase skills are not top notch :P but it works, will try to squash

[AdriVanHoudt]

ok that's better ^^ force push 🎉

[geek]

@AdriVanHoudt can't beat a good force push

[AdriVanHoudt]

@geek the one advantage in the war with git 💪 😄

[lock]

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.