Development #14353
nettoyage : faire hériter wcs/wf/timeout_jump.py du jump.py
Début:
17 décembre 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
TimeoutWorkflowStatusItem est obsolète mais encore utilisé dans un certain nombre de sites. (grep TimeoutW */workflows/* | wc -l
→ 104). Plutôt qu'avoir du code identique à celui de jump.py, l'action pourrait plutôt hériter.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-workflows-make-timeout-jump-a-thin-wrapper-of-normal.patch 0001-workflows-make-timeout-jump-a-thin-wrapper-of-normal.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 7 ans
Ack
(test lancé avec succès sans le patch sur timeout_jump.py, et ensuite aussi... je me dis que si on voulait montrer qu'on fait un peu de TDD et qu'on a bien écrit ici un test de non-regression, il pourrait y avoir deux commits, d'abord le test puis le patch sur timeout_jump.py, mais bon, ça va comme ça hein)
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
Yep, et j'ai d'abord exécuté le test avec l'ancien code. Mais j'envoie le tout dans un seul commit quand même.
commit 37eb958d1c51f90452d7efbac4d219687d471e07 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Dec 17 14:03:25 2016 +0100 workflows: make timeout jump a thin wrapper of normal jump (#14353)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: make timeout jump a thin wrapper of normal jump (#14353)