search

slipsoft / slipdb

Distributed index based search engine.
4 stars 0 forks source link

Fusion Table et TableHandler #66

Closed n-peugnet closed 5 years ago

n-peugnet commented 5 years ago

Hello, ne mergez pas tout de suite, c'est juste parce que j'ai commencé à faire cette fusion de la mort. Je vais continuer à pusher là dessus.

Vous pouvez venir m'aider sur cette branche aussi (@SylvainLune si t'es chaud)

SylvainJoube commented 5 years ago

Wow alors juste attention attention attention, ma branche symaster2 n'est pas mergée avec master et j'ai 14 commits d'avance, donc si on pouvait tout commit et tout fusionner (code d'Etienne, mon code et tes commits en attente) avant de passer à du refactoring, ça serait cool et ça nous économiserait peu-être un jour de merge (en tout cas, à moi, c'est certain)

SylvainJoube commented 5 years ago

Donc perso, je suis très chaud pour le fait de faire du refacto et virer les warinings (sur Woria, par exemple, y a aucun warning, le code est propre et je sais que c'est très important !), mais juste pas maintenant alors qu'il y a des commits en attente de merge ^^' Sinan, ça risque d'être p***** de chiant à merge, encore une fois !

n-peugnet commented 5 years ago

Pour illustrer mon commentaire sur l'autre PR

yes ce merge là va probablement être chiant. L'autre ça devrait mieux aller. Hésite pas à te faire des tests de merge dans des branches dupliquées pour voir ce que ça donne

Originally posted by @n-peugnet in https://github.com/slipsoft/slipdb/pull/65#issuecomment-481024588

merge-trkl