style: Format code with fmt #48

kosssi · 2024-08-08T15:04:31+02:00

kosssi commented

2024-08-08 15:04:31 +02:00

Détails

fmt permet de formater le code Rust, je pense que c'est une bonne chose que l'on utilise l'utilitaire aevc la commande cargo fmt nous pourrons mettre en place un test de validation des DAs quand nous aurons une CI avec la commande suivante cargo fmt --all -- --check.

Pourquoi ?

Pour rendre le code plus lisible

Documentation

https://rust-lang.github.io/rustfmt/

### Détails `fmt` permet de formater le code Rust, je pense que c'est une bonne chose que l'on utilise l'utilitaire aevc la commande `cargo fmt` nous pourrons mettre en place un test de validation des DAs quand nous aurons une CI avec la commande suivante `cargo fmt --all -- --check`. ### Pourquoi ? Pour rendre le code plus lisible ### Documentation https://rust-lang.github.io/rustfmt/

👍 1

kosssi added 1 commit 2024-08-08 15:04:32 +02:00

style: Format code with fmt 0eaf238735

kosssi requested review from florian_briand 2024-08-08 15:04:37 +02:00

kosssi requested review from theo 2024-08-08 15:04:40 +02:00

kosssi requested review from julien.misiak 2024-08-08 15:04:43 +02:00

theo approved these changes 2024-08-08 15:07:35 +02:00

kosssi merged commit 668a91941b into main

2024-08-08 15:08:03 +02:00

kosssi deleted branch fmt

2024-08-08 15:08:03 +02:00

kosssi referenced this issue from a commit

2024-08-08 15:08:04 +02:00

Merge pull request 'style: Format code with fmt' (#48) from fmt into main

florian_briand commented

2024-08-08 15:39:03 +02:00

Ça aurait valu le coup d'automatiser ça dans un githook client
et voir si on pouvait ajuster les règles, pour qu'elles soient un chouilla + aligné avec notre formatage (entre autre le passage à la ligne des into_... ) et éviter un diff "si gros" (bon, pas tant, en vrai :p mais bof en terme de lisibilité)

Ça aurait valu le coup d'automatiser ça dans un githook client et voir si on pouvait ajuster les règles, pour qu'elles soient un chouilla + aligné avec notre formatage (entre autre le passage à la ligne des `into_... `) et éviter un diff "si gros" (bon, pas tant, en vrai :p mais bof en terme de lisibilité)

theo commented

2024-08-08 15:56:03 +02:00

Pour les into_response, j'ai l'impression qu'appeler explicitement la fonction n'est pas necessaire: https://github.com/djc/askama/blob/main/askama_axum/tests/basic.rs

Ca pourrait etre une solution? @florian_briand

PS: j'ai, par hasard, trouve l'endroit dans la doc ou il specifie ca -> https://djc.github.io/askama/integrations.html#axum-integration

Pour les into_response, j'ai l'impression qu'appeler explicitement la fonction n'est pas necessaire: https://github.com/djc/askama/blob/main/askama_axum/tests/basic.rs Ca pourrait etre une solution? @florian_briand PS: j'ai, par hasard, trouve l'endroit dans la doc ou il specifie ca -> https://djc.github.io/askama/integrations.html#axum-integration

👍 1

florian_briand commented

2024-08-08 16:03:36 +02:00

Pour les into_response, j'ai l'impression qu'appeler explicitement la fonction n'est pas necessaire: https://github.com/djc/askama/blob/main/askama_axum/tests/basic.rs

Ca pourrait etre une solution? @florian_briand

Cool, bon à savoir

> Pour les into_response, j'ai l'impression qu'appeler explicitement la fonction n'est pas necessaire: https://github.com/djc/askama/blob/main/askama_axum/tests/basic.rs > > Ca pourrait etre une solution? @florian_briand Cool, bon à savoir

kosssi commented

2024-08-10 17:04:12 +02:00

Voici une PR pour ce dont tu parlais @theo P4Pillon/Krys4lide#51

Voici une PR pour ce dont tu parlais @theo https://forge.p4pillon.org/P4Pillon/Krys4lide/pulls/51

Sign in to join this conversation.

No Reviewers

florian_briand

julien.misiak

theo

3 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: P4pillon/Krys4lide#48