Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modification de la CI #8

Merged
merged 2 commits into from Nov 1, 2022
Merged

Modification de la CI #8

merged 2 commits into from Nov 1, 2022

Conversation

ghost
Copy link

@ghost ghost commented Oct 1, 2022

J'ai fait une fourchette du dépôt pour modifier l'intégration continue afin qu'elle vérifie correctement les défauts relevés par clippy. Jusqu'à maintenant, elle se contentait d'afficher des avertissements 😛

J'ai aussi retiré les commandes qui venaient modifier la version de Rust. Comme ça, le résultat est plus consistant entre les exécutions, c'est une version 1.64.0 de Rust et c'est tout. J'ai donc corrigé les derniers défauts qui se sont déclarés entre la version de Rust que j'utilise 1.58.0 et celle de l'intégration continue

PierreGagelin added 2 commits October 1, 2022 11:31
- Continuous integration was just warning about clippy defaults. We now deny them
- Applied a fix version of Rust by using an unmodified Github runner
@ghost
Copy link
Author

ghost commented Oct 1, 2022

@jblestang j'ai fini ton putain de clippy::pedantic !

Edit : pas question de faire les clippy::nursery. On pourrait faire les clippy::cargo par contre, mais j'ai un peu la flemme de me lancer là-dedans 😛

@jblestang
Copy link
Collaborator

Je suis étonné que des “unwrap” non testés passent Clippy … Time to be even more pedantic!

https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used

clippy nursery sort généralement pas mal de faux positifs … méfiance donc!

@ghost
Copy link
Author

ghost commented Oct 1, 2022

Je suis étonné que des “unwrap” non testés passent Clippy … Time to be even more pedantic!

https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used

Ce n'est pas lié à Applicability: Unresolved ? Je pense qu'ils ne veulent pas empêcher de faire du quick-and-dirty.

clippy nursery sort généralement pas mal de faux positifs … méfiance donc!

Oui, c'est pour ça que je disais "pas question" 😉

@rene-d rene-d merged commit 76499b7 into rene-d:main Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants