search

Florian-DELRIEU / HOI4_Calculator

Land battle simulator for Heart of Iron IV and Warhammer 40k
1 stars 0 forks source link

GPT - Reverting conflicted commits #61

Closed Florian-DELRIEU closed 2 months ago

Florian-DELRIEU commented 2 months ago

Résumé par Sourcery

Refactoriser la classe BattleWindow pour éliminer l'utilisation de la classe Camp, simplifiant ainsi la gestion des fonctionnalités liées au camp et améliorant la maintenabilité du code.

Améliorations :

Original summary in English ## Summary by Sourcery Refactor the BattleWindow class to eliminate the use of the Camp class, simplifying the handling of camp-related functionality and improving code maintainability. Enhancements: - Remove the Camp class and refactor the BattleWindow class to handle camp-related functionality without using the Camp class.
sourcery-ai[bot] commented 2 months ago

Guide du Réviseur par Sourcery

Cette pull request supprime la classe Camp et refactorise la classe BattleWindow pour gérer directement les fonctionnalités liées au camp. Elle simplifie la structure du code et élimine les abstractions inutiles.

Modifications au Niveau des Fichiers

Changement Détails Fichiers
Suppression de la classe Camp et des fonctionnalités associées
  • Suppression de l'importation de la classe Camp depuis BattleWindow.py
  • Suppression de la définition de la classe Camp dans Class.py
  • Suppression des instances de Camp (self.Camp_A et self.Camp_B) de BattleWindow
  • Remplacement de Camp.leader par des instances directes de StringVar (self.leader_a et self.leader_b)
  • Suppression des références à Camp.frame
 BattleWindow.py
Class.py
Refactorisation de la fonctionnalité d'ajout de division
  • Modification de la méthode add_division pour prendre le frame comme paramètre au lieu de l'instance Camp
  • Mise à jour des appels de la méthode add_division dans les commandes des boutons
  • Suppression de camp.division_list.append(division) de la méthode add_division
 BattleWindow.py
Simplification de la logique de la barre de progression dans la méthode add_division
  • Suppression de la clause else inutile dans l'affectation de la valeur de la barre de progression
  • Simplification de l'affectation de max_value pour les barres de progression
 BattleWindow.py
Conseils - Déclenchez une nouvelle révision Sourcery en commentant `@sourcery-ai review` sur la pull request. - Continuez votre discussion avec Sourcery en répondant directement aux commentaires de révision. - Vous pouvez modifier vos paramètres de révision à tout moment en accédant à votre [tableau de bord](https://app.sourcery.ai): - Activer ou désactiver le résumé de la pull request généré par Sourcery ou le guide du réviseur; - Changer la langue de révision; - Vous pouvez toujours [nous contacter](mailto:support@sourcery.ai) si vous avez des questions ou des commentaires.
Original review guide in English ## Reviewer's Guide by Sourcery This pull request removes the Camp class and refactors the BattleWindow class to handle camp-related functionality directly. It simplifies the code structure and removes unnecessary abstractions. ### File-Level Changes | Change | Details | Files | | ------ | ------- | ----- | | Removal of Camp class and related functionality |
  • Removed Camp class import from BattleWindow.py
  • Deleted Camp class definition in Class.py
  • Removed Camp instances (self.Camp_A and self.Camp_B) from BattleWindow
  • Replaced Camp.leader with direct StringVar instances (self.leader_a and self.leader_b)
  • Removed references to Camp.frame
| `BattleWindow.py`
`Class.py` | | Refactoring of division addition functionality |
  • Modified add_division method to take frame as a parameter instead of Camp instance
  • Updated add_division method calls in button commands
  • Removed camp.division_list.append(division) from add_division method
| `BattleWindow.py` | | Simplification of progress bar logic in add_division method |
  • Removed unnecessary else clause in progress bar value assignment
  • Simplified max_value assignment for progress bars
| `BattleWindow.py` | ---
Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.
Florian-DELRIEU commented 2 months ago

Reverting commit for resolve conflict with 3.1.X branch