Reboot Integration #17
Conversation
| return communes.get(codeInsee); | ||
| return { | ||
| commune: communes.get(codeInsee), | ||
| communesLoad: communes.size |
There was a problem hiding this comment.
Oulà c'est complexe. Je crois pas que ce soit justifié d'altérer cette fonction.
|
Il reste à tester les cas particulier des villes à arrondissement et les tests de l'intégration devrait être terminé. |
|
On va aussi mettre @ThibautGery comme ça on aura plusieurs sons de cloches ;) |
| const debug = require('debug')('integration'); | ||
| const centroid = require('turf-centroid'); | ||
| const streamify = require('stream-array'); | ||
| const integrate = require('./lib/integrate').integration; |
There was a problem hiding this comment.
Tu devrais rendre tout ça homogène, ainsi que les scripts dans le package.json.
integrate partout ça me semble pas mal.
There was a problem hiding this comment.
Tu voudrais remplacer tous les "integration" par '"integrate"? C'est l'idée
There was a problem hiding this comment.
Au moins à cet endroit, car tu renommes la variable du coup.
Et sinon tu peux faire :
const { integrate } = require('./lib/integrate');
|
|
||
| // Init paths | ||
| const paths = { | ||
| communesFilePath: options.communesFilePath || __dirname + '/data/communes-dp25.json', |
There was a problem hiding this comment.
Ça ne marchera pas.
Mets plutôt : __dirname + '/../data/communes-dp25.json'
|
Sinon good job ! ;) |
|
|
||
| // Init paths | ||
| const paths = { | ||
| communesFilePath: options.communesFilePath || __dirname + '../data/communes-dp25.json', |
| // Init paths | ||
| const paths = { | ||
| communesFilePath: options.communesFilePath || __dirname + '../data/communes-dp25.json', | ||
| codesPostauxFilePath: options.codesPostauxFilePath || __dirname + '../data/laposte_hexasmal.json', |
Réf #16