Skip to content

Chapitre 3 : corrections et mises à jour#97

Closed
DirtyF wants to merge 11 commits intothom4parisot:masterfrom
DirtyF:ch3-corrections
Closed

Chapitre 3 : corrections et mises à jour#97
DirtyF wants to merge 11 commits intothom4parisot:masterfrom
DirtyF:ch3-corrections

Conversation

@DirtyF
Copy link
Copy Markdown
Collaborator

@DirtyF DirtyF commented Jan 29, 2017

  • corrections de quelques typos
  • Mise à jour de quelques dépendances tel qu'indiqué dans le chapitre (npm outdated, npm install --save package@latest), vérification des exemples
  • correction d'exemples

@DirtyF DirtyF requested a review from thom4parisot January 29, 2017 20:09
Copy link
Copy Markdown
Owner

@thom4parisot thom4parisot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci beaucoup @DirtyF :-) Beau boulot !

Ça serait plus facile à merger si c'était découpé en 3 PR :

  1. le texte (0 boulot de ma part hormi la relecture donc rapide à merger)
  2. l'exemple fs-extra.js (faire tourner le code qui a changé)
  3. les dépendances (je dois passer sur le README de chaque dépendance et leurs exemples pour m'assurer que c'est OK)

Ça te dérange pas ?

Merci :-)

Comment thread chapter-03/package.json
"bluebird": "^2.10.0",
"browserify": "^11.2.0",
"async": "^2.1.4",
"babel": "^6.5.2",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

babel a beaucoup changé entre la 5 et la 6 donc si jamais y'a des exemples qui se basent dessus, ils sont à revoir – notamment au niveau des presets.

'use strict';

const { partial, compose } = require('lodash');
const { partial, flowRight } = require('lodash');
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

compose n'existe plus dans lodash@4 ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci 👍

Comment thread chapter-03/index.adoc
Le clonage du dépôt pour être remplacé par des méthodes alternatives, de type empaquetage sous forme de fichier DEB, de module Puppet, d'archive ZIP… ou encore d'archive Node via la commande `npm pack`.

Ces méthodes permettent entre autre d'inclure les dépendances contenues dans le répertoire _node_modules_ et de garantir la version exacte de l'intégralité de l'arbre de dépendance.
Ces méthodes permettent entre autres d'inclure les dépendances contenues dans le répertoire _node_modules_ et de garantir la version exacte de l'intégralité de l'arbre de dépendance.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Faut accorder même dans le cas où il s'agit de l'expression entre autre ?

Copy link
Copy Markdown
Collaborator Author

@DirtyF DirtyF Jan 29, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

entre autres = entre autres choses
source : www.academie.fr/entre-autre-ou-entre-autres/

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bon sang, 33 ans que je fais la faute… MERSSI :-D

Comment thread chapter-03/index.adoc
Tous les fichiers sauf ceux correspondant aux motifs déclarés dans les fichiers `.gitignore` et `.npmignore`.

Pour résumer, si le module est nommé `nodebook` et est en version `1.3.0`, `npm publish` enregistrera cette version dans le registre npm.
Pour résumer, si le module est nommé `nodebook` et est en version `0.4.0`, `npm publish` enregistrera cette version dans le registre npm.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bien vu !

Comment thread chapter-03/index.adoc Outdated

[TIP]
.[RemarquePreTitre]#Outil# David et Greenkeeper
.[RemarquePreTitre]#Outil# David et Greenkeeper
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La tabulation est en trop ici. Faudrait l'enlever.

Comment thread chapter-03/index.adoc Outdated
==== Critères de sélection d'un module

La sélection d'un module est un choix subjectif : la qualité, la pertinence et la popularité sont corrélé sans pour autant être des critères impératifs. +
La sélection d'un module est un choix subjectif : la qualité, la pertinence et la popularité sont corrélés sans pour autant être des critères impératifs. +
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La tabulation est en trop ici aussi (sauf si ça déclenche un effet de présentation en particulier ?)

Comment thread chapter-03/index.adoc Outdated

Compatible Node et navigateurs Web. +
[URL]#https://npmjs.com/lodash#
[URL]#https://www.npmjs.com/package/lodash#
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'utilise les URLs courtes car c'est plus facile à lire/écrire sur clavier lors d'une lecture sur ouvrage papier :-) Faudrait les laisser telles quelles.

Copy link
Copy Markdown
Collaborator Author

@DirtyF DirtyF Jan 29, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'aurais du m'en douter 😞

@DirtyF DirtyF mentioned this pull request Jan 29, 2017
@DirtyF DirtyF closed this Jan 29, 2017
@DirtyF DirtyF deleted the ch3-corrections branch January 29, 2017 21:20
@DirtyF DirtyF mentioned this pull request Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants