pierre.gauthier/Dory
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: ask and verify user inputs #1

Merged
fabien.amarger merged 2 commits from feature/getting-user-input into main 2025-02-14 14:42:23 +01:00
Conversation 0 Commits 2 Files Changed 2 +258
pierre.gauthier commented 2025-01-31 16:25:20 +01:00
Owner
Copy Link
No description provided.
pierre.gauthier added 1 commit 2025-01-31 16:25:20 +01:00
pierre.gauthier added 1 commit 2025-02-14 14:22:24 +01:00
fabien.amarger approved these changes 2025-02-14 14:42:08 +01:00
fabien.amarger left a comment
Collaborator
Copy Link

Pour moi c'est bon :) juste des petites remarques mais rien d'alarmant.
Merci pour ta contribution

Pour moi c'est bon :) juste des petites remarques mais rien d'alarmant. Merci pour ta contribution
dory_sort_out.py
@ -0,0 +71,4 @@
def create_folders(is_plu: bool, prefixe, suffixe):
"""Crée l'arborescence"""
main_path = prefixe + "_" + ("PLU" if is_plu else "PLUi") + "_" + suffixe
fabien.amarger commented 2025-02-14 14:34:53 +01:00
Collaborator
Copy Link
main_path = f"{prefixe}_PLU{"i" if not is_plu else ""}_{suffixe}"
``` main_path = f"{prefixe}_PLU{"i" if not is_plu else ""}_{suffixe}" ```
pierre.gauthier commented 2025-02-14 15:26:06 +01:00
Author
Owner
Copy Link

je note la syntaxe, c'est un bon réflexe à prendre

je note la syntaxe, c'est un bon réflexe à prendre
dory_sort_out.py
@ -0,0 +171,4 @@
+ ".pdf",
)
os.rename(file, renamed)
shutil.move(renamed, sort_path + "/" + FOLDERNAMES[4])
fabien.amarger commented 2025-02-14 14:40:33 +01:00
Collaborator
Copy Link

Ca c'est risqué, si tu ajoutes une entrée dans FOLDERNAMES ton indice ne sera plus bon

Ca c'est risqué, si tu ajoutes une entrée dans `FOLDERNAMES` ton indice ne sera plus bon
pierre.gauthier commented 2025-02-14 15:25:35 +01:00
Author
Owner
Copy Link

Je me le suis permis parce que FOLDERNAMES est une constante et ne devrais pas changer à l'avenir, mais c'est vrai que mettre une const à la place du 4 serait plus propre

Je me le suis permis parce que FOLDERNAMES est une constante et ne devrais pas changer à l'avenir, mais c'est vrai que mettre une const à la place du 4 serait plus propre
dory_sort_out.py
@ -0,0 +241,4 @@
error_report(donees_fichiers)
# Chargement
# for i in range(100):
fabien.amarger commented 2025-02-14 14:40:59 +01:00
Collaborator
Copy Link

tu peux supprimer cette partie j'imagine

tu peux supprimer cette partie j'imagine
fabien.amarger merged commit 3ecb096271 into main 2025-02-14 14:42:23 +01:00
fabien.amarger deleted branch feature/getting-user-input 2025-02-14 14:42:23 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fabien.amarger
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: pierre.gauthier/Dory#1
No description provided.
Delete Branch "feature/getting-user-input"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?