Retire les usages d'une méthode obsolète de Factory Boy #6541
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Arnaud-D
added
C-Back
|
Comment est-ce que tu actives les DeprecationWarning ? J'essaie de reproduire et comprendre le problème que tu rencontres. |
|
Oui, j'ai oublié de préciser. C'est la sortie de cette commande qui m'aide à inspecter les avertissements : On passe tous les tests avec un paramètre pour avoir tous les warnings et je transfère le tout vers un fichier pour lire ça tranquillement ensuite. En regardant la doc, je vois qu'on peut faire moins de warning avec quelque chose du genre : mais ça ne change rien au fait que ce que je change dans cette PR soit obsolète. |
philippemilink
requested changes
Oct 12, 2023
Arnaud-D
force-pushed
the
resout-obsolescence-factoryboy
branch
from
9925496
to
288bd7e
Compare
* remplace _after_postgeneration par une méthode custom * conserve le comportement important pour notre code * anticipe le prochain changement de version majeur * facilite la lecture des logs d'obsolescence en évitant plus de 10,000 lignes liées à cette méthode
Arnaud-D
force-pushed
the
resout-obsolescence-factoryboy
branch
from
288bd7e
to
ea3200c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dans le cadre de la préparation pour Django 4, j'analyse les logs d'obsoléscence à la recherche de DeprecationWarning et autres choses du genre. Un des problèmes est qu'une méthode remplit plus 10000 lignes de logs (plus de la moitié), ce qui complique le reste.
Cette méthode est une de Factory Boy :
_after_postgeneration.Cette PR :
Contrôle qualité
Comme c'est une méthode utilisée seulement dans les tests, la CI devrait suffire.