Job tech improve promise usage

octo-webf / octo-job-board

https://jobs.octo.com/#/

6 stars 7 forks source link

Job tech improve promise usage #162

Closed clementdubois closed 6 years ago

clementdubois commented 6 years ago

Question : quelle version de node en prod ? (async/await à partir de la 7 uniquement) => http://node.green/ réponse : A priori en 7, c'est pas mal mais peut mieux faire, on peut passer la version de node en 8 (LTS) voir en 9 ?

Warning : j'ai fait une confiance aveugle aux tests :) Un peu de refacto côté serveur:

Utiliser async/await au lieux d'une chaine de promesse qui stocke des valeurs intermédiaire
Utiliser map au lieu de reduce quand reduce n'est pas nécessaire
Utiliser lodash plutôt que des implem native de fonction de parcours de tableau (flattenDeep, difference)
Ne pas faire de Promise.resolve pour le return dans un then (par défault une valeur retourné dans un then est enrobé par un Promise.resolve)
Utiliser la déstructuration de variable pour affecter des variables à partir des clé d'un objet

coveralls commented 6 years ago

Coverage decreased (-57.9%) to 37.255% when pulling 45651b79f4599dc78a91a12564eea11920f0b49d on JOB-tech-improve-promise-usage into ec833a4780c8d8ed984224ad790cb1bd9f739784 on dev.

coveralls commented 6 years ago

Coverage decreased (-0.5%) to 94.713% when pulling 1a90918f6d0dfd56e83b51b4da32669afc809ee8 on JOB-tech-improve-promise-usage into ec833a4780c8d8ed984224ad790cb1bd9f739784 on dev.

trollepierre commented 6 years ago

v6.11.5 => Mais on peut upgrade (ça ne posera pas de problème - moi je code en v8)

coveralls commented 6 years ago

Coverage decreased (-0.5%) to 94.713% when pulling bac755c59e5f4e1d67b08e7bf34f867f8954d626 on JOB-tech-improve-promise-usage into ec833a4780c8d8ed984224ad790cb1bd9f739784 on dev.

trollepierre commented 6 years ago

J'ai eu des problèmes lorsque je suis passé en 9.1.0 Donc je tente la version 8.9.1

coveralls commented 6 years ago

Coverage decreased (-0.5%) to 94.713% when pulling 120b5721853053763c4af4fd98eae7c4cc251ae1 on JOB-tech-improve-promise-usage into ec833a4780c8d8ed984224ad790cb1bd9f739784 on dev.

coveralls commented 6 years ago

Coverage decreased (-57.9%) to 37.255% when pulling a1d1475ff3b23302463834b0e5730b2f6200806e on JOB-tech-improve-promise-usage into ec833a4780c8d8ed984224ad790cb1bd9f739784 on dev.

coveralls commented 6 years ago

Coverage decreased (-57.9%) to 37.255% when pulling c6b4220191b29a8616ef4ca8cc3516477494b281 on JOB-tech-improve-promise-usage into ec833a4780c8d8ed984224ad790cb1bd9f739784 on dev.

coveralls commented 6 years ago

Coverage decreased (-0.007%) to 95.161% when pulling 37d758a27d5e56ecdc14598dfffd5642532528ad on JOB-tech-improve-promise-usage into ec833a4780c8d8ed984224ad790cb1bd9f739784 on dev.