Nettoyage du code

[Gabriel29306]

🚀 Nouvelle Pull Request

Proposez vos modifications pour améliorer Papillon

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

• Vous avez testé de build le projet avec vos modifications et ce build a réussi
• Vous respectez les conventions de codage et de nommage du projet
• Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
• Cette pull request n'est pas un duplicata d'une autre
• Cette pull request est prête à être revue (review) et fusionnée (merge)
• Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
• Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
• Les détails des changements ont été décrits ci-dessous
• Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

• Retrait d'une grande partie des variables inutilisées (je crois avoir tout fait)
  • J'ai uniquement laissé une ou deux variables où leur utilités étaient avérées, ou qu'il y avait un message expliquant qu'il y aurait une utilité future.
  • En faisant ça, j'ai aussi dégagé le code qui était les dépendances uniques de ces variables
• Changement de plusieurs keywords de variables utilisant var par let (J'aurais peut être dû utiliser des const à quelques endroits)
• Retrait de code considéré comme "inaccessible"
  • Tel que des break après des return
• Retrait de code inutile tel que des void, return ou await sur des fonctions non async.
• Ajout d'un paramètre dans tsconfig.json pour spécifier que c'est bien du code react native.
• Application de toutes les règles ESLint recommandéss

Informations supplémentaires

D'autres PR de ce type sont à prévoir prochainement, sauf si elles ne sont pas considérées comme intéressantes.
Ce genre de PR peut avoir l'air violente car elle fait surtout du retrait mais elle permet de simplifier une bonne partie du code.

[Kgeek33]

Faut voir comment on peut s'occuper dans App.tsx avex le no-duplicate-imports pour @/background/BackgroundTasks

pour App.tsx, pas le choix, ajoute /* eslint-disable-next-line no-duplicate-imports */ à un des import pour qu'il n'y ait pas d'erreurs

[Kgeek33]

@Gabriel29306

• mets en resolve le commentaire suivant => Nettoyage du code #689 (comment)
• résous les conflits

une fois fait, j'approuverai, j'viens de recheck, tout est bon pour moi

[Kgeek33]

C'est bon pour moi !

[Copilot]

Pull Request Overview

This pull request cleans up the codebase by removing unused variables and unreachable code while enforcing consistent coding standards. Key changes include the removal of redundant break statements and unused state variables, conversion from var to let for better scoping, and updates to ESLint configuration and custom rules to enforce code quality.

Reviewed Changes

Copilot reviewed 285 out of 285 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/components/Global/FileIcon.tsx	Removed unreachable break statements after return calls
src/components/FirstInstallation/PapillonShineBubble.tsx	Removed unused state variable (sHeight) and its setters
src/components/Addons/AddonsWebview.tsx	Unified log handling into a single conditional for various log types
src/background/Notifications.ts	Simplified boolean assignments by removing unnecessary ternaries
src/background/BackgroundTasks.ts	Refactored event handling to use log function and proper scoping
src/addons/addons.ts	Replaced var with let for improved scoping
plugins/notifee-mod.js	Minor code style adjustments for arrow function parameters
eslint_rules/*	Added custom ESLint rules for detecting redundant logical and ternary expressions
eslint.config.js	Updated ESLint configuration to include the new custom rules
App.tsx	Improved code clarity with formatting and duplicate import handling

[Gabriel29306]

Faudrait que je fasse par plusieurs PR au lieu de tout faire en une 🤔

[Kgeek33]

Faudrait que je fasse par plusieurs PR au lieu de tout faire en une 🤔

Ouais, ça me rappelle les pr sur Expo 52, un commit => conflit 😂😂

• PR 1 => supprimer tous les fichiers inutilisés (il y en a plein)
• PR 2 => faire comme cette pr mais par dossier présent dans src
• PR i (#maths_expertes) => configurer ESLint

[Gabriel29306]

Faut que j'y réfléchisse, je résoud les conflits régulièrement, mais bon, ça ralenti les reviews

[Gabriel29306]

Par contre le résumé de Copilot est trop juste.

[Kgeek33]

Si tu veux une review plus rapide et éviter de te péter le crâne pour résoudre les conflits, sépare cette pr en plusieurs

• Une pr = un dossier
• Dossier en priorité : views