Résolution des problèmes de storage, re-formattage avec fmt et clippy #13

Merged

Nex8192 merged 4 commits from snowcode/cygnus-bot:dev into dev

2023-02-03 10:16:05 +01:00

snowcode commented

2023-02-03 09:55:51 +01:00

Member

Désolé c'est un peu le bordel mais mon éditeur re-formatte automatiquement les fichiers que je modifie du coup je pouvais pas faire simplement plusieurs pull requests.

En résumé :

file_storage.rs

Ce que j'ai fait c'est qu'à la ligne du map_or_else j'ai remplacé None par || None car la fonction prends une closure et pas une Option
Puis dans le write, j'ai remplacé role.to_le_bytes().as_ref() par role.to_string().as_bytes() car sinon le .parse qu'on fait quand on le lit plante et la fonction retourne toujours None
Et enfin à la toute fin du storage dans les tests j'ai remplacé Some(1047839857575219212) par Some(&1047839857575219212) car sinon on a un Some(&u64) attendu mais a trouvé Some(u64)
Suppression tous les imports qui ne servent à rien pour virer un max de warnings
Enfin, bonus clippy, on peut remplacer tous les &self.base_directory.blablabla par self.base_directory.blablabla

error_storage.rs

Remplacement de tous les blablablaFailure par blablabla comme conseillé par clippy

set_role.rs

Retourne une erreur si il y a eu un problème au niveau du stockage

.gitignore

Ignore le dossier storage pour éviter d'ajouter les fichiers de storage dans le git

Tout le reste

Utilisation de la commande cargo fmt pour le formattage
Utilisation de la commande cargo clippy --fix pour la résolution automatique de certains warnings

Désolé c'est un peu le bordel mais mon éditeur re-formatte automatiquement les fichiers que je modifie du coup je pouvais pas faire simplement plusieurs pull requests. En résumé : ## file_storage.rs - Ce que j'ai fait c'est qu'à la ligne du `map_or_else` j'ai remplacé `None` par `|| None` car la fonction prends une closure et pas une `Option` - Puis dans le write, j'ai remplacé `role.to_le_bytes().as_ref()` par `role.to_string().as_bytes()` car sinon le .parse qu'on fait quand on le lit plante et la fonction retourne toujours None - Et enfin à la toute fin du storage dans les tests j'ai remplacé `Some(1047839857575219212)` par `Some(&1047839857575219212)` car sinon on a un `Some(&u64)` attendu mais a trouvé `Some(u64)` - Suppression tous les imports qui ne servent à rien pour virer un max de warnings - Enfin, bonus clippy, on peut remplacer tous les `&self.base_directory.blablabla` par `self.base_directory.blablabla` ## error_storage.rs - Remplacement de tous les `blablablaFailure` par `blablabla` comme conseillé par clippy ## set_role.rs - Retourne une erreur si il y a eu un problème au niveau du stockage ## .gitignore - Ignore le dossier `storage` pour éviter d'ajouter les fichiers de storage dans le git ## Tout le reste - Utilisation de la commande `cargo fmt` pour le formattage - Utilisation de la commande `cargo clippy --fix` pour la résolution automatique de certains warnings

snowcode added 4 commits

2023-02-03 09:55:52 +01:00