search

DraftBot-A-Discord-Adventure / DraftBot

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

Non null assertion issues in the code #2537

Open BastLast opened 1 month ago

BastLast commented 1 month ago

JS-0339 desactivated in deepsource because it was blocking everything but we should discuss maybe of ways to improve that, this issues is made for that

romain22222 commented 1 month ago

si j'ai bien compris l'idée derrière ce check, en gros, c'est l'équivalent d'asserts en python : si ca passe l'assertion, rien se passe, si ca la passe pas, ca soulève une erreur

Je pense qu'il faudra refact, on ne cherche pas ce comportement dans la grande majorité des cas, si y'a un soucis, on voudrait plutôt qu'elle soit relevée, et à la limite qu'on envoie à l'utilisateur qu'il y a eu une erreur, avec potentiellement un code qu'on génèrerait qui quand nous on le rentre dans la db, nous renvoie le log d'erreur. Ce serait plus efficace pour diagnostiquer les problèmes qui pourraient survenir.

romain22222 commented 1 month ago

mais bon, pas prioritaire, je pense c'est une bonne idée d'ignore pour le moment

Feiryn commented 1 month ago

L'idée c'est plutôt de dire au compilateur que tu es sûr que la valeur à cet endroit là n'est pas null ou undefined. Vu qu'on est en mode strict, s'il y a un doute sur un null ou undefined, ça ne compilera pas

BastLast commented 1 month ago

L'idée c'est plutôt de dire au compilateur que tu es sûr que la valeur à cet endroit là n'est pas null ou undefined. Vu qu'on est en mode strict, s'il y a un doute sur un null ou undefined, ça ne compilera pas

On est pas en mode strict je crois ya une autre issue pour passer en mode strict

Feiryn commented 1 month ago

Les modules Discord et Lib sont en strict

BastLast commented 1 month ago

Fuck je confond v4 et v5 mdr