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

Supprime des fonctions inutiles #6547

Merged

Conversation

Arnaud-D
Copy link
Contributor

C'était auparavant un morceau de #6441, mais ça sera plus facile pour la revue de découper un peu.

J'ai utilisé la recherche de Pycharm pour vérifier l'absence d'utilisation de ces fonctions dans le code.

Contrôle qualité

La CI devrait suffire.

@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest labels Oct 28, 2023
@Arnaud-D Arnaud-D changed the title Supprime fonctions inutiles Supprime des fonctions inutiles Oct 28, 2023
@coveralls
Copy link

Coverage Status

coverage: 88.807% (+0.004%) from 88.803% when pulling 241a03b on Arnaud-D:supprime-fonctions-inutiles into 0a733a7 on zestedesavoir:dev.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La CI est OK, j'ai vérifié à coups de git grep que les fonctions n'étaient pas utilisées (mais aussi les versions dans les get_ comme il me semble que c'est possible dans les templates), tout m'a l'air OK.

@philippemilink philippemilink merged commit 59a44a2 into zestedesavoir:dev Nov 1, 2023
12 checks passed
@Arnaud-D Arnaud-D deleted the supprime-fonctions-inutiles branch November 1, 2023 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants