`report_internal_error` does not respect `stdout` and `stderr` in several places
まだ誰も着手していません。
- 主要言語
- Python
- スター
- 20.6k
- フォーク
- 3.3k
- 平均マージ
- 1日 18時間
- マージ済み PR(30日)
- 54
説明
When using mypy as a library and passing stdout and stderr into build.build it is reasonable to expect that it will be respected for all cases.
But, there are several places where report_internal_error does not respect the passed value and just dumps everything into sys.stdout:
But, there also places where report_internal_error is called with the correct streams:
I suggest to add stdout and stderr props to Options class. It would be possible to easily work with them from all these three missed places.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、build.build から Options クラスを経由する stdout と stderr の流れを追い、次に checker.py、checkexpr.py、semanal.py、build.py にある report_internal_error の呼び出しを調べます。3 つの見落とされていた呼び出し箇所が sys.stdout ではなく渡されたストリームを使用し、build.py の既存の動作を維持できれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- devtools
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 45/100