`warnings.deprecated` has no `skip_file_prefixes`
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Feature or enhancement
Proposal:
Seems like when adding warnings.deprecated in https://github.com/python/cpython/pull/104004, it was forgotten to add one of the two stack skipping arguments.
It needs it for the same reasons why it already got stacklevel, e.g. because adding decorators adds stack levels in a hard to predict way (e.g. a 3rd party decorator might start calling the wrapped function one level deeper, suddenly changing where the warning points)
Has this already been discussed elsewhere?
Yes, and it was accepted, but then apparently fogotten, see link below
cc @brettcannon @JelleZijlstra
Links to previous discussion of this feature:
https://discuss.python.org/t/pep-702-marking-deprecations-using-the-type-system/23036/11
Linked PRs
- gh-156937
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Examinez l’implémentation et les tests de warnings.deprecated, en commençant par la manière dont stacklevel est géré et par la manière dont skip_file_prefixes est représenté ailleurs. Le PR lié gh-156937 est le travail actuel à examiner. Le travail est considéré comme terminé lorsque warnings.deprecated prend en charge skip_file_prefixes pour les cas indiqués concernant le décorateur et stack-trace, avec des tests couvrant le comportement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100