pylint-dev/pylint

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

Open

#10,092 opened on 2024年11月22日

GitHub で見る
 (4 comments) (0 reactions) (1 assignee)Python (4,978 stars) (1,059 forks)batch import
False Negative 🦋Good first issueHacktoberfestHelp wanted 🙏Needs PR

説明

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

コントリビューターガイド

技術スタック
python
領域
tooling
Issue 種別
research
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
active
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Pythonpylint checker APIos.environ usage
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
50
調査方針
Investigate the checker in pylint/checkers/stdlib.py around line 33 to understand the current implementation of invalid envvar default. Check if os.environ.get is intentionally excluded or if it's an oversight. Review any comments on this issue (issue #10092) for maintainer input. Consider adding support for os.environ.get by extending the function that checks envvar defaults.