WIP: Counter/127 replace all panics by fatal error

Closed #127 (closed)

Tien c'est un reliquats que j'ai oublié ça, on utilise plus la feature log_panics, tu peut virer le if et ne garder que le code dans le else ;)

changed this line in version 2 of the diff

added 10 commits

• 497cbcbc...101f305e - 6 commits from branch dev
• c8d907df - [ref] common-tools fatal_error() now returns ! to handle panic
• 821b29fd - [ref] conf/lib.rs: any call to panic!() that do not have more than 1 argument...
• fbb15ef3 - [ref] common-tools/src/lib.rs: useless feature to log error removed
• 9d4d026d - [ref] more panic!() replaced by fatal_error()

Compare with previous version

@librelois Dans modules/blockchain/blockchain/src/sync/download/json_reader_worker.rs ligne 79 j'ai eu des problèmes avec unreachable!();. Je l'ai retiré. J'espère aussi ne pas trop avoir modifié la logique du code.

Merci counter, j'ai bien vu ta notif, je regarderai çà demain soir :)

Étant très peu dispo, il peut se passer du temps avant que je puisse reviewer :/

@counter-reverse : review: ko

Le nommage des commit ne respecte pas les conventions du projet: https://git.duniter.org/nodes/rust/duniter-rs/blob/dev/doc/fr/developpeurs/conventions-git.md#nommage-des-commits

commit 1 : il manque le caractère ':' après la portée commit 2 : la portée doit correspondre au nom de la crate, il ne faut pas aller jusqu'au fichier commit 3 : idem + description trop longue et trop vague, préféréz "rm use of old feature log_panics" commit 4 : tu a oublié la portée

Relis tes nom de commit avec une gui git avant de pusher stp, ces détails devraient te sauter aux yeux :)

D'accord je vais renommer les commits. C'est juste le nom des commits qui pose problème? Le code est fonctionnel? S'il n'y avait pas le nom des commits est-ce que on aurait pu l'intégrer?

@counter-reverse je ne sais pas car je n'ai pas encore testé. Mon temps est trop rare et précieux en ce moment donc je ne teste une contribution que lorsque je n'ai plus de retours avant tests :)

closed