16612Fermer16614
Arvi89Le 08/02/2018 à 15:04
Kevin Kofler (./16612):
Arvi89 (./16606) :
Il ne pouvait pas reproduire l'erreur car après 12 jours, il n'avait toujours pas une installation similaire (c'est quand même leur logiciel, c'est le minimum de pouvoir installer ce que tu proposes à tes clients), et d'ailleurs il y a eu 0 test sur leur modif, c'est un fichier directement lié au stockage qui a été modifié, il manquait un return...
Ils ont dit qu'ils feront plus de tests, mais 2 semaines pour finalement avoir un fix par un utilisateur sur un bug ultra bloquant, c'est moyen.
Le correctif vient de la même personne qui a envoyé le commit fautif (icewind1991).
Oui mais ça n'aurait pas dû être accepté en premier lieu, on n'accepte pas un PR sans la tester (et ils ont vraiment lu vite fait aussi, il n'y avait que 2 lignes de modifiées, ils n'ont pas vu qu'il manquait le return). Ça fait un peu peur sur ce qu'ils acceptent du coup, j'utilise ça pour héberger mes données perso quand même.