forumNouveau sujet

Je passe en revue le code qu'on a récupéré — existe-t-il une checklist de revue de code pour les non-développeurs ?

EErcan T***Membre actifMembre de la communauté
Membre depuis
avr. 2022
Message
138
#1

nous sommes une entreprise de 14 personnes basée en Californie et fournissant un logiciel logistique et nous nous sommes séparés de notre ancienne agence ; ils viennent de livrer la dernière phase d'un portail client sur mesure un projet de huit mois pour lequel nous avons payé un total de 65.000 dollars... nous avons transféré le dépôt de code sur notre propre compte mais nous n'avons aucun développeur en interne et j'ai moi-même un profil purement commercial / gestion.

nous avons signé avec une nouvelle équipe de freelances mais avant de verser le dernier paiement de 10.000 dollars à l'ancienne agence, je tiens à comprendre par moi-même dans quel état se trouve ce que nous avons reçu et je sais bien que je ne pourrai pas fouiller le code ligne par ligne pour traquer les errurs de logique mais existe-t-il une checklist concrète de revue de code source qu'un manager non technique pourrait vérifier visuellement (structure des fichiers documentation rigueur générale des livrables) ? quelles questions poser lors de la recette et que chercher dans le dépôt ?

LLeyla Y***Membre actifMembre de la communauté
Membre depuis
mars 2026
Message
61
Plus utile#2

En bref : vous ne pourrez pas juger seul de la qualité du code à la ligne près, mais vous pouvez tout à fait vérifier, même sans bagage technique, que le dépôt est complet, autonome et conforme aux standards professionnels. Avant de débloquer le paiement final, il est impératif de confirmer que le projet tourne sans dépendre de tiers.

Commencez votre checklist par ces étapes : 1) Présence du guide d'installation. Le dépôt doit contenir un 'README' ou une doc d'installation. Donnez ce document à un dev externe à l'agence qui ne connaît rien au projet et demandez-lui de monter le système sur une machine vierge en suivant uniquement ces étapes. S'il n'arrive pas à le faire tourner sans appeler l'agence, la livraison n'est pas terminée. 2) Vérif licence et droits d'auteur. Le code ne doit pas contenir de structures sous licence privée copiées d'autres clients de l'agence, de libs commerciales expirées ou de licences open source interdites.

3) Scan des mots de passe et clés en dur. Tapez des termes comme 'password', 'secret', 'api_key' dans la barre de recherche du code. Vérifiez que les vrais mots de passe de base de données ou les clés de paiement en prod ne sont pas écrits en clair directement dans le code, mais appelés via des variables d'environnement. 4) Historique des modifs et propriété. Regardez si les commits du dépôt sont faits avec des messages propres ; des centaines de fichiers entassés sur un seul jour avec des messages d'une ligne genre 'version finale', 'correction' montrent que l'agence cache son processus de dev. Ne validez pas le paiement sans avoir fait examiner et confirmer ces quatre points de base par votre nouvelle équipe avec un PV de livraison officiel.

SSelin K***Membre actif
Poste
Chargé de ressources humaines
Secteur
E-commerce
Type d'organisation
atelier
Membre depuis
sept. 2024
Message
42
#3

Le point le plus critique, c'est l'autonomie. Le code peut-il tourner sur votre propre serveur sans aucune dépendance envers un serveur externe de l'agence, leurs comptes développeurs privés ou les e-mails personnels de leurs employés ? Si un service de notifications tiers ou une clé de serveur est rattaché à leur compte, votre système se retrouvera bloqué tôt ou tard.

KKader A***Membre actif
Poste
Directeur de magasin
Secteur
Cosmétique
Type d'organisation
entreprise à deux succursales
Membre depuis
nov. 2022
Message
183
#4

Même sans compétences techniques, vous pouvez lancer des outils d'analyse automatique de code. Activez le scanner de sécurité gratuit proposé par votre plateforme de dépôt. Ces outils génèrent un rapport listant les failles évidentes et les dépendances obsolètes ; vous n'avez plus qu'à transmettre ce document à l'agence en leur demandant de tout nettoyer.

ÖÖzgür B***Membre actifMembre de la communauté
Membre depuis
févr. 2023
Message
34
#5

L'année dernière, on a fait exactement la même erreur sur une passation similaire à 40.000 dollars. On pensait avoir récupéré le code, mais aucun test automatisé n'avait été écrit. La nouvelle équipe nous a demandé 12.000 dollars et 6 semaines supplémentaires juste pour comprendre le système et rédiger les tests. Vérifiez impérativement s'il y a un dossier de tests et s'il est bien rempli.

ZZafer S***Membre actif
Poste
Employé de magasin
Secteur
Assurance
Type d'organisation
entreprise individuelle
Membre depuis
déc. 2025
Message
67
#6

Passer le dépôt en revue à l'œil nu en tant que manager ne vous donnera qu'un faux sentiment de sécurité. Vous ne détecterez pas une fuite de mémoire dissimulée ou une requête SQL mal optimisée simplement en lisant des noms de fichiers. Demander à votre nouvelle équipe un audit technique écrit de passation pour 500 à 1.000 dollars reste une solution bien plus fiable.

AAslıhanMembre actif
Poste
Fabricant de mode
Membre depuis
avr. 2024
Message
102
#7

Dès demain, prévoyez un partage d'écran d'une heure avec votre nouveau freelance. Demandez-lui d'installer le système de zéro, de le lancer, de restaurer une sauvegarde de la base de données et d'effectuer une action depuis le panneau d'administration. S'il y parvient sans accroc, vous pourrez payer le solde l'esprit tranquille.

CCaner B***Membre actifMembre de la communauté
Membre depuis
déc. 2024
Message
1
#8

Quand notre ancienne agence nous a livré le projet, tout semblait parfait. Deux mois plus tard, lors d'une redirection de nom de domaine, on s'est rendu compte que l'intégration des notifications tierces était reliée à la carte bancaire personnelle du patron de l'agence. Dès que sa carte a expiré, plus rien ne fonctionnait. Assurez-vous que tous les comptes de services tiers ont bien été transférés sur des adresses e-mail appartenant à votre entreprise.

BBeren V***Membre actif
Poste
Technicien de maintenance
Secteur
Publicité et promotion
Type d'organisation
Entreprise de 120 personnes
Membre depuis
mars 2024
Message
123
#9

regardez aussi les dates des derniers commits sur le repo selon moi. s'ils ont poussé 1000 fichiers d'un coup à deux jours de la fin du contrat c'est possible que même eux ne sachent plus trop ce qu'il y a dedans et qu'ils aient tout bâclé pour boucler.

PS : c'est demandé plus bas, j'ai répondu dans le deuxième message.

MMurat B***Nouveau membre
Poste
Agent du service client
Secteur
Produits de la mer
Type d'organisation
coopérative
Membre depuis
juin 2026
Message
123
#10

Je vous conseille de vérifier les clauses de cession des droits de propriété intellectuelle dans votre contrat. Même si le code vous a été remis, des litiges juridiques sur la propriété peuvent survenir en l'absence d'un protocole de cession et de quitus formellement signé. La validation technique et le transfert juridique doivent être menés de front.

AAhmet N***Expert
Poste
Employé de magasin
Secteur
Construction
Type d'organisation
filiale d'un groupe
Membre depuis
juil. 2022
Message
153
#11

Désolé, mais ce n'est pas vrai dans tous les cas. N'hésitez pas à demander, celui qui ne demande pas paie toujours plus cher.

Mettre en place un processus de demande de changement n'ralentit pas le travail, il l'accélère. J'espère que cela vous sera utile.

HHüseyin Ş***Membre actif
Poste
Administrateur réseau
Secteur
Emballage
Type d'organisation
filiale d'un groupe
Membre depuis
juil. 2023
Message
81

Doki · Identité de marque · 2025

#12

ne manquez rien : Tous ceux qui se pressent sur checklist revue de code source se heurtent au même obstacle.

c'est mon avis je ne l'écris pas comme une vérité absolue.

UUğur D***Membre actif
Poste
Saisie de données
Secteur
Services de santé
Type d'organisation
entreprise individuelle
Membre depuis
janv. 2022
Message
2
#13

J'ai été soulagé en lisant cette réponse, donc ce n'est pas seulement mon cas. Tout va bien les trois premiers mois, les problèmes arrivent au quatrième.

Bien sûr, cela change si votre situation est différente.

ZZehra Y***Membre actif
Poste
Directeur marketing
Secteur
Fabrication de meubles
Type d'organisation
Entreprise de 20 personnes
Membre depuis
mai 2024
Message
324
#14

La réponse ci-dessus va droit au but. Si le périmètre s'agrandit il faut augmenter soit le délai soit le budget. Il n'y a pas de troisième option.

Je suis aussi curieux de savoir si d'autres font autrement.

KKader A***Nouveau membre
Poste
Administrateur réseau
Secteur
Conseil
Type d'organisation
entreprise de taille moyenne
Membre depuis
sept. 2026
Message
10
#15

C'est vrai en théorie, mais ça ne marche pas ainsi en pratique. Tout va bien les trois premiers mois, les problèmes arrivent au quatrième.

Corrigez-moi si je me trompe.

NNazlı G***Membre actif
Poste
Comptable
Secteur
Agriculture
Type d'organisation
entreprise de taille moyenne
Membre depuis
nov. 2023
Message
176
#16

Je vais essayer.

CCem I***Membre actifMembre de la communauté
Membre depuis
sept. 2025
Message
4
#17

La discussion part dans tous les sens, je recentre. Si les critères d'acceptation ne sont pas écrits, la date de fin du projet est ouverte à discussion.

UUfuk S***Membre actif
Poste
Comptabilité préliminaire
Secteur
Agriculture
Type d'organisation
coopérative
Membre depuis
juin 2022
Message
74

Doki · Support à la réponse aux incidents · 2025

#18

J'écris cela pour éviter que vous ne fassiez la même erreur. Les accès doivent être créés au nom de l'entreprise, pas avec des comptes personnels.

Avant de décider, regardez quelles données vous avez en main.

DDamla M***Membre actif
Poste
Représentant commercial terrain
Secteur
Immobilier
Type d'organisation
entreprise de taille moyenne
Membre depuis
juil. 2025
Message
63

Doki · Application mobile · 2023

#19

Désolé mais ce n'est pas vrai dans tous les cas. L'offre la moins chère est généralement la moins réfléchie.

Corrigez-moi si je me trompe.

ÖÖzge A***ExpertMembre de la communauté
Membre depuis
avr. 2025
Message
42
#20

Je me permets une petite mise en garde. Avant de décider, regardez quelles données vous avez en main.

À votre place, j'irais par cette voie.

Répondre