Première PR, nettoyage cosmétique

[GillesDebunne]

Cette Pull Request est

• [x] Un correctif
• [ ] Une nouvelle fonctionnalité

Description

Première PR sur cette base de code.

Quelques modifications mineures et cosmétiques :

• Remplacer le type void par undefined pour les variables. D'après la page TS "Declaring variables of type void is not useful"
• Changer les quelques var utilisés par des let et const plus TS
• Supprimer le fichier delay.ts qui n'est pas utilisé
• Modifier le type NormalizedSearch

[GillesDebunne]

Ce champ __normalized_search: n’est jamais utilisé.

Il est même trompeur puisque dans la suite du code, on « cast » des string en NormalizedSearch avec des as.

[fcamblor]

Il me semble que cette notation avec le __normalizedSearch avait été introduite par @Floby et avait pour objectif de pouvoir contourner le duck typing sur le type NormalizedSearch (vu comme un simple alias de string)

=> en gros pour pas qu'on puisse mettre n'importe quelle string dans une variable de type NormalizedSearch

[Floby]

correct, grosso modo c'est de la convention ça ne change pas grand chose à part demander au compilateur de vérifier qu'on est passé par la fonction normalize pour la variable qu'on manipule

[GillesDebunne]

J'ai remis en place le type string & { _:undefined } qui évite effectivement le duck typing. J'ai commenté.

J'ai uniformisé l'usage de Normalized à la place de Searchable.

J'ai aussi éclairci les commentaires autour de la méthode toFullTextNormalized partagée.

[fcamblor]

Au-delà de la petite typo que j'ai relevé (et que je te laisse confirmer), c'est OK pour moi. @GillesDebunne tu voulais itérer encore un peu sur d'autres amélioration ou je merge ? :)

[GillesDebunne]

Ok pour merger

Et pour corriger la typo (ou je le ferai en rentrant cette aprem)

Merci !

[fcamblor]

@GillesDebunne merci à toi pour la PR :)