search

geo2france / idg-qgis-plugin

Plugin QGIS pour la consultation des données des différentes Infrastructure de Données Géographiques en France
GNU General Public License v2.0
14 stars 4 forks source link

Déplacer des fichiers Python hors de toolbelt #70

Open bchartier opened 2 months ago

bchartier commented 2 months ago

Je me demande si le répertoire toolbelt a vocation à contenir certains de nos fichiers Python comme browser.py, tree_node.py, remote_platforms.py... J'ai l'impression qu'il a été fait pour contenir du code utilitaire qui n'est pas spécifique au plugin. J'aurais tendance à mettre ces fichiers dans un autre répertoire pour que ce soit plus clair.

jbdesbas commented 2 months ago

Je link avec #66 (organisation du code)

Guts commented 2 months ago

Une des pratiques et de stocker ce qui est lié à la logique propre à un programme dans un dossier core ou logic. toolbelt étant un peu comme utils mais moins "conflictuel" (on trouve du utils partout) et contient plutôt du code réutilisable d'un projet similaire à l'autre (ici un plugin QGIS).

my 2 cents

bchartier commented 2 months ago

Une des pratiques et de stocker ce qui est lié à la logique propre à un programme dans un dossier core ou logic. toolbelt étant un peu comme utils mais moins "conflictuel" (on trouve du utils partout) et contient plutôt du code réutilisable d'un projet similaire à l'autre (ici un plugin QGIS).

Merci pour cet éclairage. Ça m'arrange bien car c'est cohérent avec ma proposition de sortir ce qui est vraiment spécifique de ce projet du dossier toolbelt. E n plus j'ai le sentiment que cela réduirait le risque d'imports cycliques.

bchartier commented 1 month ago

Discussion d'aujourd'hui avec @jbdesbas : déplacer les actions de plugin_actions.py vers le dossier gui.