Update code sample on about page with ES6 features

[Chris911]

Small update on the about page to use ES6 features in code sample supported on on Node latest.

Looks like:

[fhemberger]

Nice, LGTM

[rvagg]

maybe it's time to do a proper job and introduce template strings at the same time?

consthttp=require('http');http.createServer((req,res)=>{res.writeHead(200,{'Content-Type': 'text/plain'});res.end('Hello World\n');}).listen(1337,'127.0.0.1',function(){console.log(`Server running at http://${this.address().address}:${this.address().port}/`);});

or is that too complicated?

[fhemberger]

@rvagg Good point! Maybe add:

constconnection=this.address();console.log(`Server running at http://${connection.address}:${connection.port}/`);

[ghost]

LGTM, regarding the complexity, maybe link to the ES6 page right below the code block? something like This code example uses EcmaScript 2015 (ES6) features. Read more _here_

[Chris911]

@rvagg, @fhemberger, I can make the change the add the template string but personally I think it adds complexity by using this.address().address. Since this is a small introduction snippet maybe something like:

consthttp=require('http');constHOSTNAME='127.0.0.1';constPORT=1337;http.createServer((req,res)=>{res.writeHead(200,{'Content-Type': 'text/plain'});res.end('Hello World\n');}).listen(PORT,HOSTNAME,function(){console.log(`Server running at http://${HOSTNAME}:${PORT}/`);});

This also matches the terminology used in the server.listen documentation.

[fhemberger]

@Chris911 Good idea, let's settle for this.

[Chris911]

@fhemberger updated.

[stevemao]

👍

[phillipj]

Last nitpick, use arrow function on this callback aswell?

[Chris911]

Good catch.

[rvagg]

I'd vote for lower-case variable names, there's inconsistency in http vs PORT and HOSTNAME and upper-casing constants is not necessarily idiomatic Node or even JavaScript even though it's common in other languages.

[Chris911]

Fine with me @rvagg.

[Chris911]

Updated.

[phillipj]

Nice 👍