python / python/mypy

Add support for attr.converters.default_if_none

Offen
#6,136 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

feature priority-1-normal topic-attrs
Vorherrschende Sprache
Python
Sterne
20.6k
Forks
3.3k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

I have the following:

@attr.s(auto_attribs=True)
class Foo:
    bars: List[Bar] = attr.ib(
        default=attr.Factory(list),
        converter=default_if_none(default=attr.Factory(list)),
    )

I'm running mypy via pre-commit and getting:

error: Unsupported converter, only named functions and types are currently supported

I have the following in my .pre-commit-config.yaml:

- repo: https://github.com/pre-commit/mirrors-mypy
  rev: v0.650
  hooks:
  - id: mypy
    additional_dependencies:
      - attrs>=18.2.0

(Without the additional_dependencies part I also get error: Module 'attr.converters' has no attribute 'default_if_none')

Could the issue be that there's special handling for optional here but not for default_if_none?

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

Beginne in mypy/plugins/attrs.py bei der im Issue verknüpften bestehenden Sonderbehandlung für optionale Konverter und reproduziere dann das gemeldete Snippet mit mypy über die pre-commit-Konfiguration. Verfolge, wie attr.converters.default_if_none validiert wird, und ermittle das erwartete Typisierungsverhalten; als erledigt gilt die Aufgabe, wenn der Konverter ohne die gemeldeten Fehler akzeptiert wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

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

Neue Issues direkt in Ihr Postfach

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