Replace var with let or const as appropriate.

[curran]

I'm curious, what the stance is in terms of adopting let and const in this repo?

Possible approaches:

• Consistently ES5 (var only)
• Consistently ES6 (e.g. would a PR be welcome that converts all var to let & const?)
• Allow internal inconsistency (mixed var, let, const within & between files)

This relates to how PRs should be reviewed with respect to usage of these keywords. Thanks!

[Fil]

I don't know, but I like the precision and code safety (and scoping) given by let/const. However it's true that it could be a long time during which the code base would be inconsistent.

If we wanted to semi-automate the migration process we could use lebab (converse of babel). See the experiment I did on d3-geo at d3/d3-geo#180 — some good came out of it, in particular a 3% smaller file size. If you look at the commits you can see the lebab "transforms" applied one by one, to help figure out a bit better what's possible. (There were suprisingly few manual steps.)

[curran]

Nice! lebab looks like a great approach. Inconsistent state seems fine - thanks for confirming.

[domoritz]

I think if the goal is to converge towards ES6, some internal inconsistency is fine. Maybe you can have something in the guidelines to ask people to gradually update to ES6 in their pull requests.

Moving everything at once is usually only something that core developers can do since you need very quick merges (to avoid conflicts) and some knowledge of how things fit together.

[curran]

Let's lebab d3/d3-*/**/*.js!

Here's a question: would PRs be welcome that just migrate individual files to ES6?

Maybe one file (or few files) at a time is the safest way to do it.

[mbostock]

It’s not a goal at this time to rewrite everything. We’re focused on getting 6.0 released.

[Fil]

@domoritz has done the manual work of replacing var declarations with the appropriate const and let for d3-scale (d3/d3-scale#212). One issue is that the resulting (compressed) file size has increased by 1%.

To be clear I think it's a good idea to have the more precise code; we just have to be a bit cautious. Some changes allowed by ES6 syntax are also going to bring gains in download times.

[domoritz]

Is the 1% more after or before gzip compression?

To me, 1% more is a good trade off for safer code. Also, you could run Babel over the code to remove the let/const in the released code.

[Fil]

Ah, right! I had measured the minified's size—but you're correct that what we should measure is the gzip'ed minified: then it's not even 1%, but 0.4% "heavier" 👍

[domoritz]

Great. To me, that's within the noise.

[anlexN]

please rewrite d3 in es2015+ syntax

[curran]

lol PRs welcome.

[domoritz]

I sent one here: d3/d3-scale#212 ;-)