search

DraftBot-A-Discord-Adventure / DraftBot

A Discord adventure bot that speaks english and french
MIT License
222 stars 62 forks source link

Vérifier l'organisation des fichiers de constante de la v5 #2426

Closed BastLast closed 3 months ago

BastLast commented 4 months ago 
          faudrait qu'on passe en revue l'emplacement des constantes avant la fin de la v5 parce que j'ai l'impression qu'il y en a  qui sont mal placées dans des fichiers de constantes génériques (ex lib a un fichier de constantes générique et des fichiers classés dans des dossiers..)

_Originally posted by @BastLast in https://github.com/DraftBot-A-Discord-Adventure/DraftBot/pull/2425#discussion_r1504604455_

BastLast commented 4 months ago

pour simplifier le developpment toutes les constantes sont déplacées dans lib, peut etre un refacto aura lieu plus tard

BastLast commented 3 months ago

je pense que c'est une bonne v1, yavais 2 constantes qui devaient absolument être dans core car elles importaient des trucs de core donc pour le moment je les ai mises là bas avec un todo un peu à l'arrache donc ça sera couvert dans #2429

BastLast commented 3 months ago

je pense que pour une v1 on est pas mal, si on veut faire plus, on ouvrira une nouvelle issue