Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Déplacer des fichiers Python hors de toolbelt #70

Closed
bchartier opened this issue Jul 19, 2024 · 5 comments · Fixed by #88
Closed

Déplacer des fichiers Python hors de toolbelt #70

bchartier opened this issue Jul 19, 2024 · 5 comments · Fixed by #88
Assignees
Labels
code Related to code organization, refactoring, legibility, best practices enhancement New feature or request

Comments

@bchartier
Copy link
Collaborator

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.

@bchartier bchartier added the question Further information is requested label Jul 19, 2024
@bchartier bchartier self-assigned this Jul 19, 2024
@jbdesbas
Copy link
Member

Je link avec #66 (organisation du code)

@Guts
Copy link

Guts commented Jul 25, 2024

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
Copy link
Collaborator Author

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 bchartier added enhancement New feature or request and removed question Further information is requested labels Aug 7, 2024
@bchartier bchartier added the code Related to code organization, refactoring, legibility, best practices label Aug 22, 2024
@bchartier
Copy link
Collaborator Author

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

@bchartier
Copy link
Collaborator Author

J'ai déplacé la dernière suggestion dans un autre ticket car ce n'est pas directement lié à toolbelt : #89

@bchartier bchartier linked a pull request Oct 10, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code Related to code organization, refactoring, legibility, best practices enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants