Proposition d'améliorations majeures pour la branche v2
Bonjour,
Tout d'abord, félicitations pour ce projet et pour la réécriture en Python sur la branche feat/v2
, c'est un excellent travail.
En explorant le code, j'ai eu quelques idées pour peaufiner certains aspects et j'ai pris la liberté de préparer un patch qui, je l'espère, pourra être utile. Il s'agit bien sûr d'une proposition, n'hésitez pas à l'utiliser en partie ou en totalité.
Voici un résumé des modifications proposées :
1. Implémentation de la sauvegarde pour "Toujours utiliser pour cette URL"
- Suggestion : Finaliser la logique de l'interrupteur présent dans l'interface pour permettre aux utilisateurs de sauvegarder leurs choix.
-
Dans le patch : La sauvegarde est fonctionnelle. L'application crée et met à jour les fichiers de configuration dans
~/.config/fireflex/
lorsqu'un utilisateur choisit de toujours ouvrir une URL avec un navigateur spécifique.
2. Ajout d'une base de tests avec Pytest
- Suggestion : Mettre en place un environnement de test pour solidifier la base de code et faciliter les contributions futures.
-
Dans le patch :
pytest
est ajouté comme dépendance de développement. Un répertoiretests/
est créé avec un premier test poururlutils.py
, confirmant que l'environnement est prêt.
3. Gestion des erreurs de lancement de navigateur
- Suggestion : Améliorer l'expérience utilisateur en cas d'échec du lancement d'un navigateur.
-
Dans le patch : Si
gtk-launch
échoue, une boîte de dialogue d'erreur s'affiche désormais, informant l'utilisateur du problème au lieu d'échouer silencieusement.
4. Mise à jour de la documentation (README) et ajout des Docstrings
-
Suggestion : Mettre à jour le
README.md
pour qu'il corresponde à la nouvelle version Python et ajouter des docstrings pour améliorer la maintenabilité. -
Dans le patch : Le
README.md
a été réécrit pour refléter la nouvelle procédure d'installation avec Poetry. Des docstrings ont été ajoutées à toutes les méthodes et classes principales.
Comment tester ces changements ?
Toutes ces modifications sont regroupées dans un unique fichier patch pour faciliter la revue et l'application.
-
Télécharger le patch :
wget https://gitlab.villejuif.fr/-/snippets/13/raw/main/fireflex-v2-improvements.patch -O fireflex-v2-improvements.patch
-
Appliquer le patch :
git checkout feat/v2 git apply fireflex-v2-improvements.patch
J'espère que cette contribution vous sera utile. Encore bravo pour ce projet !%
Edited by Barikad