Skip to content

Chapitre3 dependances#99

Closed
DirtyF wants to merge 24 commits intothom4parisot:masterfrom
DirtyF:chapitre3-dependances
Closed

Chapitre3 dependances#99
DirtyF wants to merge 24 commits intothom4parisot:masterfrom
DirtyF:chapitre3-dependances

Conversation

@DirtyF
Copy link
Copy Markdown
Collaborator

@DirtyF DirtyF commented Jan 29, 2017

Annule et remplace #97

@ThomasG77
Copy link
Copy Markdown

ThomasG77 commented Feb 13, 2017

@DirtyF Ta correction sur le remplacement de compose est la + fidèle au code original mais une autre solution comme ci-dessous est envisageable si on ne veut pas embarquer tout lodash (intéressante surtout si on a un usage côté client).

const partial = require('lodash.partial');
const compose = require('lodash.compose');

Par contre, pas sûr que cette solution "bis" que je soumet n'embrouille pas plus un lecteur débutant car cela demande plus d'explications et apporte potentiellement un risque de confusion...

@thom4parisot
Copy link
Copy Markdown
Owner

thom4parisot commented Feb 21, 2017

@DirtyF j'ai refait un tour des changements proposés et je vais reventiler le chapitre 3 l'orienter sur la création d'une webapp avec Express et tout. Donc la majorité des exemples et de la progression va changer. Donc les dépendances aussi.

Ça permettra d'avoir 1 chapitre historique, 1 chapitre rappels et design pattern, 1 chapitre webapp, 1 chapitre frontend, 1 chapitre CLI etc. – là y'a trop de redondances entre le 2 et le 3.
Comme ça on se débarasse de #64 qui est un exemple que je n'ai qu'à moitié fini et dont je ne retrouve plus le sens ni l'intention aujourd'hui :-D

@DirtyF
Copy link
Copy Markdown
Collaborator Author

DirtyF commented Feb 21, 2017

@oncletom ça fait une chouette progression ouais, c'est mieux ainsi.

@DirtyF DirtyF closed this Feb 21, 2017
@DirtyF DirtyF deleted the chapitre3-dependances branch February 21, 2017 14:01
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.

4 participants