DraftBot-A-Discord-Adventure / DraftBot

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

refactor: badgeCommand to ts and slash command #1017

Closed BastLast closed 2 years ago

romain22222 commented 2 years ago

needs a review

Feiryn commented 2 years ago

Faudrait enlever le yarn.lock de tes commits

romain22222 commented 2 years ago

Faudrait enlever le yarn.lock de tes commits

draft m'avait dit de le commit, j'ai suivi ses dires, mais du coup ok je ne le commit pas next time

BastLast commented 2 years ago

Faudrait enlever le yarn.lock de tes commits

draft m'avait dit de le commit, j'ai suivi ses dires, mais du coup ok je ne le commit pas next time

Faudrait faire un truc pr ca normalement quand on met à jour le package.jsonn on met aussi à jour le yarn lock, ou alors il faut le gitignore

Feiryn commented 2 years ago

Ton yarn.lock c'est juste que ton IDE change le format donc y a rien qui change. Sinon je sais plus s'il est nécessaire de le garder pour que ça fonctionne bien

BastLast commented 2 years ago

Ton yarn.lock c'est juste que ton IDE change le format donc y a rien qui change. Sinon je sais plus s'il est nécessaire de le garder pour que ça fonctionne bien

normalement le yarn lock il faut le commit, en tout cas perso on m'a toujours dit ça après c'est vrais que ca n'a rien à faire dans cette pr spécifiquement faudrait faire ça a part

https://stackoverflow.com/questions/39990017/should-i-commit-the-yarn-lock-file-and-what-is-it-for

Feiryn commented 2 years ago

Ton yarn.lock c'est juste que ton IDE change le format donc y a rien qui change. Sinon je sais plus s'il est nécessaire de le garder pour que ça fonctionne bien

normalement le yarn lock il faut le commit, en tout cas perso on m'a toujours dit ça après c'est vrais que ca n'a rien à faire dans cette pr spécifiquement faudrait faire ça a part

https://stackoverflow.com/questions/39990017/should-i-commit-the-yarn-lock-file-and-what-is-it-for

C'est bien ce qu'il me semblait. Du coup oui ça n'a rien à faire là, faut le commit qu'en cas de changement de version, d'ajout ou de suppression de dépendance.

BastLast commented 2 years ago

oui ça n'a rien à faire là, faut le c

Ton yarn.lock c'est juste que ton IDE change le format donc y a rien qui change. Sinon je sais plus s'il est nécessaire de le garder pour que ça fonctionne bien

normalement le yarn lock il faut le commit, en tout cas perso on m'a toujours dit ça après c'est vrais que ca n'a rien à faire dans cette pr spécifiquement faudrait faire ça a part https://stackoverflow.com/questions/39990017/should-i-commit-the-yarn-lock-file-and-what-is-it-for

C'est bien ce qu'il me semblait. Du coup oui ça n'a rien à faire là, faut le commit qu'en cas de changement de version, d'ajout ou de suppression de dépendance.

yep du coup je propose de le virer de cette pr mais faut quand meme le maj dans une nouvelle pr à part vu qu'ila a pas été modifié lors de la derniere modif des dépendances

Feiryn commented 2 years ago

oui ça n'a rien à faire là, faut le c

Ton yarn.lock c'est juste que ton IDE change le format donc y a rien qui change. Sinon je sais plus s'il est nécessaire de le garder pour que ça fonctionne bien

normalement le yarn lock il faut le commit, en tout cas perso on m'a toujours dit ça après c'est vrais que ca n'a rien à faire dans cette pr spécifiquement faudrait faire ça a part https://stackoverflow.com/questions/39990017/should-i-commit-the-yarn-lock-file-and-what-is-it-for

C'est bien ce qu'il me semblait. Du coup oui ça n'a rien à faire là, faut le commit qu'en cas de changement de version, d'ajout ou de suppression de dépendance.

yep du coup je propose de le virer de cette pr mais faut quand meme le maj dans une nouvelle pr à part vu qu'ila a pas été modifié lors de la derniere modif des dépendances

🤔 Il a été modifié si

BastLast commented 2 years ago

ah ya forcément un moment ou il a été oublié sinon il serait pas édit lors d'un nouveau yarn install

Ou alors j'ai rien compris ?

Feiryn commented 2 years ago

Il a été modif car le pc de romain a reformaté tout le fichier, si tu regardes le contenu y a des quotes en plus, des lignes qui sont inversées etc...

BastLast commented 2 years ago

ok bah nique alors