Yup, I've been doing that as I go. I just haven't felt it was important enough to be tackled just for the sake of it. Consider that it will create quite some diff noise and probably merge conflicts for any open PRs.
That being said, there's a tool that does these transformations for you: https://lebab.io/. Last time I checked it was a bit overzealous with the arrow functions, resulting in bad code layout. And some transformed choked on our license header. So *if* we do this, it might be worth considering throwing something like `prettier` at the code to format it uniformly (and make sure we make `git blame` completely useless). @janpio Would love to do that, I hear :wink: Of course, for `var` to `const` simple search and replace should work too. Then step through linter errors and change these all to let. Probably gets you 99% there with the minimal impact. [ Full content available at: https://github.com/apache/cordova/issues/12 ] This message was relayed via gitbox.apache.org for [email protected]
