Tititatatoto

[npiganeau]

@rayman520 Tu peux pousser stp ? Parce que tu marques les pb resolved, mais moi je ne vois rien :-)

[npiganeau]

J'ai résolu les conflits, il faut que tu pull à nouveau la branche tititatatoto chez toi pour récupérer les modifs.

J'ai refait le tour et j'ai vu 2 trucs qui bloquent encore:

1. Le module i18nUpdate: j'aime pas le nom. En Go, le nom du module doit être un truc qui fait que la fonction qu'on appelle dans un autre module avec le package en préfixe donne un truc sympa. Par exemple: imageTools.NewImage() c'est nul alors que image.New() c'est classe (!). Dans notre cas: i18nUpdate.RegisterPoUpdateRuleSet() ou i18nUpdate.RegisterPoUpdateFunc() c'est bof. Je n'ai pas la solution, mais on en reparle.

2. Le hexya i18n update -l all ne fonctionne pas. A partir du moment où tu as inventé le all (et c'est cool !), il faut qu'il marche dans toutes les commandes hexya

[npiganeau]

Et j'ai oublié: il faut que les tests passent. Ici tu as commité le fichier server qui appelle hexCron alors qu'il n'existe pas encore. Il faut donc que tu l'enlèves car j'aimerais faire un merge avant d'attaquer les workers.

[codecov[bot]]

Codecov Report

Merging #33 into dev will decrease coverage by 1.34%. The diff coverage is 3.47%.

@@            Coverage Diff            @@
##             dev      #33      +/-   ##
=========================================
- Coverage   83.9%   82.56%   -1.35%     
=========================================
  Files         57       57              
  Lines       6780     6894     +114     
=========================================
+ Hits        5689     5692       +3     
- Misses       820      930     +110     
- Partials     271      272       +1

Impacted Files	Coverage Δ
hexya/models/fields.go	86.21% <ø> (ø)	:arrow_up:
hexya/models/methods.go	82.95% <0%> (-1.52%)	:arrow_down:
hexya/i18n/init.go	37.5% <0%> (-22.5%)	:arrow_down:
hexya/i18n/i18n.go	45.59% <1.92%> (-51.08%)	:arrow_down:
hexya/models/fields_defs.go	84.43% <100%> (+0.03%)	:arrow_up:
hexya/models/bootstrap.go	85.86% <33.33%> (-0.34%)	:arrow_down:

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 65501d7...1f4a0fc. Read the comment docs.