pylint-dev/pylint

Use inference to determine if **kwargs is missing a named parameter

Open

#8.785 geöffnet am 18. Juni 2023

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (1.059 Forks)batch import
False Negative 🦋Good first issueHacktoberfestNeeds PR

Repository-Metriken

Stars
 (4.978 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 58T 21h) (63 gemergte PRs in 30 T)

Beschreibung

I would expect these two lines to raise the same set of messages. We don't use inference to analyze whether **kwargs contains the missing named parameter x. We could.

import copy
import os

copy.copy(**{"y": os.environ})  # [unexpected-keyword-arg]
copy.copy(y=os.environ)  # [no-value-for-parameter, unexpected-keyword-arg]

False negative for no-value-for-parameter.

Contributor Guide

Research-Richtung
Untersuche, wie pylint derzeit **kwargs in Funktionsaufrufen behandelt. Das Problem weist auf ein falsches Negativ für 'no value for parameter' hin, wenn **kwargs verwendet wird. Sieh dir den Codebase für die Inferenzlogik in Bezug auf das Entpacken von Keyword Argumenten an, wahrscheinlich im Checker für variable Argumente. Überlege, wie die Inferenz verbessert werden könnte, um fehlende benannte Parameter zu erkennen, wenn **kwargs verwendet wird. Relevante Dateien könnten pylint/checkers/typecheck.py oder ähnliche Inferenzmodule enthalten.
Tech Stack
python
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Pythonpylint internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50