Add option to enforce variable declaration at the top of declaring scope

[jscs-dev]

Add option to enforce variable declaration at the top of declaring scope

[mdevils]

Hello.

I will be glad to receive a good pull-request on this.

[mikesherov]

@Druotic, are you still interested in implementing this? If not, I can do it. Please let me know. I'll wait a few days for your answer, and assume you're not if I get no response.

[Druotic]

@mikesherov Sorry for the late repsonse. I plan to implement it eventually, but it will be some time...possible a couple of weeks from now due to various life events :( If you have time to (or already have) implemented this, be sure to mention this issue so it can be closed. Otherwise, I'll get around to it eventually.

[mikesherov]

@Druotic, still interested?

[Druotic]

Wow, I had completely forgotten about this... At this point, probably not. But, if I change my mind, I'll just check back and see if anyone has added it yet.

[justinanastos]

+1

[markelog]

This issue has been open more than a month without supported PR for it.

Unfortunately, we don't have the resources to implement rules that could not be included in any of the supported presets (if it could be, please point that out).

However, if you or anybody else submit PR for it, we will gladly consider it.

[Krinkle]

@markelog This would be a rule to enable in the Wikimedia preset. We currently enforce this rule by convention only. I'd love for JSCS to enforce it.

👍 To quote my colleges "How is this not supported yet". I guess people who write multiple var declarations don't overlap that much with people who like to use linters, or write linters anyway. Anyways glad I found something, anything on this topic.

Any preference for the rule name, I will try to take a look into this.
If anyone has any idea about an existing rule that can act as a start it will be a time saver.
Or else I would dig into the existing rules for implementation.

Think I found the requireMultipleVarDecl that might be applicable for this. I am thinking about names like requireVarDeclAtTop with the only option value of true unless someone else has better suggestions.

[mrjoelkemp]

requireVarDeclFirst sounds closer to the rule's logic.
On Jul 6, 2015 3:50 AM, "oredi" [email protected] wrote:

Think I found the requireMultipleVarDecl that might be applicable for
this. I am thinking about names like requireVarDeclAtTop with the only
option value of true unless someone else has better suggestions.

—
Reply to this email directly or view it on GitHub
#99 (comment).

[markelog]

Check this out - #807

@mrjoelkemp and @markelog Thanks I will use that and and look into that sample implementation.