Development #95476
Agendas : mutualiser le code de get_all_slots et get_free_time
0%
Description
Plusieurs blocs de code en commun dans ces deux méthodes, à factoriser donc.
Factorisation qui sera utile pour #94732.
History
Updated by Valentin Deniaud 25 days ago
Cependant get_free_time c'est utilisé seulement pour l'ANTS, c'est donc du code qui n'a pas vocation à être maintenu indéfiniment : c'est pas mal si dégager cette feature le moment venu reste simple. (mais je ne sais pas ce que tu imagines, si découper get_all_slots en plusieurs méthodes rend le tout plus lisible, pourquoi pas)
Updated by Robot Gitea 25 days ago
- Status changed from Nouveau to En cours
Emmanuel Cazenave (ecazenave) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/chrono/pulls/312
- Titre : WIP: misc: factorize some slots code (#95476)
- Modifications : https://git.entrouvert.org/entrouvert/chrono/pulls/312/files
Updated by Benjamin Dauvergne 25 days ago
De mon coté +1, j'avais pensé le faire quand j'ai pondu get_free_time() et puis flemme. Je suis d'accord que ça devrait rendre la méthode get_all_slots() plus lisible (elle pourrait même prendre un nom plus claire au passage).
Updated by Robot Gitea 11 days ago
- Status changed from Solution proposée to Solution validée
Gael Pasgrimaud (gpasgrimaud) a approuvé une pull request sur Gitea concernant cette demande :
Updated by Robot Gitea 11 days ago
- Status changed from Solution validée to Résolu (à déployer)
Emmanuel Cazenave (ecazenave) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/chrono/pulls/312
- Titre : misc: factorize some slots code (#95476)
- Modifications : https://git.entrouvert.org/entrouvert/chrono/pulls/312/files
Updated by Transition automatique 9 days ago
- Status changed from Résolu (à déployer) to Solution déployée