search

Virtual-Royaume / Royaume-Discord-Bot

It is the bot of our Discord community, it offers various utility features. It is written in TypeScript and uses DiscordJS to interact with the Discord API.
9 stars 1 forks source link

Correction/Clean du code, bonne pratique, utilisation de ESLint... #23

Closed Melvynx closed 3 years ago

Melvynx commented 3 years ago

Clean du code en général

fix: typescript @ts-ignore add: eslint to avoid let fix: eslint warning and error to have a clean projects

Todo:

Bluzzi commented 3 years ago

Pourquoi tu as mis les packages suivant en dependencie classique et dev ?

{
    "@typescript-eslint/eslint-plugin": "^4.28.1",
    "@typescript-eslint/parser": "^4.28.1"
}
Bluzzi commented 3 years ago

Je ne suis pas très fan de Prettier pour certaines mises en forme qu'il met en place (je te montrerais + tard, mais je ne pense pas que ça soit quelque chose que l'on peux corriger), es-ce possible d'effectuer avec ESLint une vérification des erreurs de syntaxe via ESLint ? Puis laisser les développeurs les corriger eux même

Melvynx commented 3 years ago

Pourquoi tu as mis les packages suivant en dependencie classique et dev ?

{
    "@typescript-eslint/eslint-plugin": "^4.28.1",
    "@typescript-eslint/parser": "^4.28.1"
}

ahahas ça c'est eslint qui à fait du bon boulot ahah (effectivement c'était faux, j'ai supp)

Je ne suis pas très fan de Prettier pour certaines mises en forme qu'il met en place (je te montrerais + tard, mais je ne pense pas que ça soit quelque chose que l'on peux corriger), es-ce possible d'effectuer avec ESLint une vérification des erreurs de syntaxe via ESLint ? Puis laisser les développeurs les corriger eux même

Oui tous est configurable

Bluzzi commented 3 years ago

Des informations sur l'avancement de cette PR ici : https://github.com/Virtual-Royaume/Royaume-Database/pull/1

Désolé pour le retard qui est pris sur ses deux PR !

Bluzzi commented 3 years ago

Je merge finalement cette PR avec ESLint et sa configuration par défault, nous verrons plus tard si nous souhaitons utiliser une configuration particulière. Mais je pense qu'utiliser la configuration de base est une façon de faire correcte.

Pour Prettier, pour l'instant nous allons continuer sans, peut-être réfléchir à utiliser la solution prête dans ESLint. Pour l'instant nous continuons manuellement.

Pour l'automatisation, nous réfléchirons dans une adoption plus poussé sur les repos du Royaume à une solution pour lancé une vérification avant chaque merge. En attendant ça sera fait manuellement et une commande npm run lint a était ajouté pour éxécuter ESLint.