atom / language-javascript

JavaScript language package for Atom
Other
194 stars 236 forks source link

use let/const es6 syntax for, forin, forof #691

Open mrbbp opened 3 years ago

mrbbp commented 3 years ago

Requirements

Description of the Change

sorry first time,

i use the for, forin and for...of snippets but my linter configured to works with es6 is unhappy. so i propose the change var to more contemporary variable declaration let and const

Alternate Designs

Benefits

more contemporary variable declaration

Possible Drawbacks

possibly confused for beginners

Applicable Issues

perhaps a prob with old navigator

apicgg commented 2 years ago

This makes sense. Thanks. @sadick254 Could you please merge this to master? It will be really helpful to have 'let' instead of 'var'.