pylint-dev/pylint

Question: Is there a reason that `invalid-envvar-default` only checks `os.getenv`, not `os.environ.get`

Open

#10.092 geöffnet am 22. Nov. 2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (1.059 Forks)batch import
False Negative 🦋Good first issueHacktoberfestHelp wanted 🙏Needs PR

Repository-Metriken

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

Beschreibung

is there a reason that invalid-envvar-default only checks os.getenv, not os.environ.get?

https://github.com/pylint-dev/pylint/blob/68cb5b320653ad64c68ff48a4bb4ba449a01d3a6/pylint/checkers/stdlib.py#L33

Contributor Guide

Research-Richtung
Untersuche den Checker in pylint/checkers/stdlib.py um Zeile 33, um die aktuelle Implementierung von invalid envvar default zu verstehen. Prüfe, ob os.environ.get absichtlich ausgeschlossen ist oder ob es sich um ein Versehen handelt. Überprüfe alle Kommentare zu diesem Issue (Issue #10092) auf Eingaben der Maintainer. Erwäge, die Unterstützung für os.environ.get durch Erweiterung der Funktion hinzuzufügen, die die Standardwerte von Umgebungsvariablen überprüft.
Tech Stack
python
Domain
developer experience
Issue Type
Recherche
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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
Python
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70