python / python/cpython

`warnings.deprecated` has no `skip_file_prefixes`

Offen
#156,929 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

extension-modules type-feature
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
35.9k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Überprüfe die Implementierung und Tests für warnings.deprecated, beginnend damit, wie stacklevel behandelt wird und wie skip_file_prefixes an anderer Stelle dargestellt wird. Der verknüpfte PR gh-156937 ist die aktuelle Arbeit, die untersucht werden soll. Als erledigt gilt, wenn warnings.deprecated skip_file_prefixes für die angegebenen Decorator- und stack-trace-Fälle unterstützt und Tests das Verhalten abdecken.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
backend
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.