pylint-dev/pylint

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

Open

#8 785 ouverte le 18 juin 2023

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)Python (1 059 forks)batch import
False Negative 🦋Good first issueHacktoberfestNeeds PR

Métriques du dépôt

Stars
 (4 978 stars)
Métriques de merge PR
 (Merge moyen 58j 21h) (63 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez comment pylint gère actuellement **kwargs dans les appels de fonction. Le problème signale un faux négatif pour 'no value for parameter' lors de l'utilisation de **kwargs. Examinez le code source pour la logique d'inférence liée au déballage des arguments par mot clé, probablement dans le vérificateur d'arguments variables. Réfléchissez à la manière dont l'inférence pourrait être améliorée pour détecter les paramètres nommés manquants lorsque **kwargs est utilisé. Les fichiers pertinents peuvent inclure pylint/checkers/typecheck.py ou des modules d'inférence similaires.
Stack technique
python
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Pythonpylint internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50