jscs-dev / node-jscs

:arrow_heading_up: JavaScript Code Style checker (unmaintained)
https://jscs-dev.github.io
MIT License
4.96k stars 513 forks source link

Add new option to requirePaddingNewLineAfterVariableDeclaration #2173

Closed AuthorProxy closed 8 years ago

AuthorProxy commented 8 years ago

What about new option to this rule, something like "allExcept": singleVar or even better something like "minRequiredVars": 1. Like this rule but only when there more than 1 var declaration, it's very disqusting to have a lot of small methods (2-3 lines) with empty lines.

markelog commented 8 years ago

Would you like to implement it?

AuthorProxy commented 8 years ago

If nobody take it, I will

hzoo commented 8 years ago

We released 3.0! We also wrote a post about how 3.0 will be our last major release and that we are merging with ESLint.

So we won't be working on any new features moving forward including new rules/options. We will continue to fix bugs and work on CST and better autofixing for current rules in the upcoming months.

I'm putting an orphaned label to note this won't be worked on.

I would encourage anyone who wants the rule to try writing it themselves: I would either write a custom rule or submit an issue to ESLint and if accepted a PR. Otherwise a custom rule that won't be in core repo itself.

JSCS rules are pretty similar to ESLint rules: here are some resources to get started: