Conversion du code vers ES2015

[thom4parisot]

fix #55

[DirtyF]

@oncletom grosse PR 😨

T'as converti avec un outil, t'as linté le code ? Faut tester que tous les codes fonctionnent ?

[thom4parisot]

J'ai converti à la main ;-) J'ai linté (travis le fait aussi).

J'ai testé quelques examples, pas tous. Je dirais qu'il faudrait juste essayer ceux qui paraissent "dangereux". Et je ferai un truc pour tester les examples à plus grande échelle plus tard (juste changer de version de Node ou une dépendance peut péter des trucs)

[gmarty]

T'aurais pu utiliser Lebab.

[DirtyF]

@oncletom Souhaites tu effectuer une transformation auto ou pas du coup ?

Y'a t-il des règles à suivre ? (utiliser seulement let et les arrow functions ? lebab est assez souple à ce niveau et se configure.

'use strict';

var cycles = 0;

while(cycles++ < 1000000000);

module.exports = cycles;

Avec Lebab :

const cycles = 0;

while(cycles++ < 1000000000);

export default cycles;

On voit que le mode strict saute, const remplace var, et que l'export de module utilise la nouvelle syntaxe. Pas mal pour 3 lignes déjà, j'imagine sur des parties plus importantes ou plus complexes.

Si tu as transformé à la main, il y a surement une bonne raison et une transformation de syntaxe sera peut-être moins pédagogique dans certains cas.

[thom4parisot]

Merci @gmarty pour suggérer lebab ! J'en avais entendu parler mais pas lu en détail la doc. Chose faite désormais.

Y'a pas mal d'éléments de conversion qui sont manquants (d'après la doc) ou unsafe. Ça mange pas de pain de sélectionner des flags de conversion, de voir ce que ça donne dans une PR et de comparer les deux.

Dans le cas de ce livre, j'ai envie de garder l'export en module CommonJS pour que le code soit interprétable par Node.js nativement. C'est bon de savoir que le use strict; n'est plus nécessaire :-)

[SebastienElet]

Le use strict; semble toujours nécessaire si tu ne passes pas par babel (qui le rajoute en transpilant).

Je viens de tester en node 6.9.4 avec les examples de mdn : https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode/Transitioning_to_strict_mode#Differences_from_non-strict_to_strict

[DirtyF]

@oncletom En ne sélectionnant que les options safe de lebab et en gardant strict et l'export de modules CommonJS, ton code est compliant 🎉

Les deux seules exceptions mineures :

chapter-02/examples/src/interrupt.js

--process.stdin.on('data', function(){});
++process.stdin.on('data', () => {});

chapter-03/examples/src/snippets/npm-cli-table.js

--table.push.apply(table, modules);
++table.push(...modules);

Ça c'est pour la syntaxe, maintenant faut encore tester avec node 6.9.4

[DirtyF]

LGTM 👍 (syntaxement parlant)

[DirtyF]

FYI ce fichier n'est pas présent et génère une erreur lors de l'exécution en ligne de commande :

❯ node chapter-03/examples/src/snippets/cli-app.js
module.js:471
    throw err;
    ^

Error: Cannot find module '../lib/models/user.js'

[thom4parisot]

Merci @DirtyF ! Je pense que j'ai bourdé et soit jamais commité models/user.js soit je pensais l'écrire et je ne l'ai jamais fait. Je le case dans une issue à part 👍

@SebastienElet ah, merci pour le complément ! Ça va me servir pour #4.