Raise errors when "# noqa" lines don't have any issues
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
まず、Flake8 が # noqa コメントをどのように処理し、無視された違反をどのように報告するかを、typing-comment の例を具体的なケースとして使って追跡します。# noqa コメントがどの問題も抑制していない場合に Flake8 がエラーを出し、有効な抑制は引き続き機能すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
In GitLab by @OddBloke on Jul 27, 2017, 06:27
Please describe how you installed Flake8
On Ubuntu Artful:
$ apt install flake8
Please provide the exact, unmodified output of flake8 --bug-report
{
"dependencies": [
{
"dependency": "setuptools",
"version": "36.0.1"
}
],
"platform": {
"python_implementation": "CPython",
"python_version": "3.5.3+",
"system": "Linux"
},
"plugins": [
{
"plugin": "mccabe",
"version": "0.6.1"
},
{
"plugin": "pycodestyle",
"version": "2.3.1"
},
{
"plugin": "pyflakes",
"version": "1.5.0"
}
],
"version": "3.2.1"
}
Please describe the problem or feature
I would like to be able to instruct flake8 to error out when a "# noqa" comment doesn't cause an error to be ignored. This would help us keep our codebase as flake'd as possible, because we'd be forced to remove linting ignores as they are fixed or become irrelevant.
As a specific example, we have code which currently causes an F401 because the imports are only used in Python typing comments. A reductive case:
from typing import Optional # noqa: F401
def foo():
# type: () -> Optional[int]
...
Once #118 has been addressed, this line should no longer be raising an F401, so I would like to be forced to remove the noqa comment. (This would then mean, after the removal, that we would correctly catch the addition of unused typing imports.)
- 主要言語
- Python
- スター
- 3.8k
- フォーク
- 363
- 平均マージ
- 2時間 53分
- マージ済み PR(30日)
- 10
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
PyCQA/flake8 のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
PyCQA/flake8#2062 · コメント 4 件 ·
メンテナーはふだん 1 日以内に返信
-
Max Line Length in style guide ignored with python 3.14 when checking multiple files再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
PyCQA/flake8#2011 · コメント 5 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
PyCQA/flake8#1793 · コメント 9 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
PyCQA/flake8#1774 · コメント 13 件 ·
メンテナーはふだん 1 日以内に返信
-
option_manager.add_option ignores action when reading from config再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
PyCQA/flake8#1770 · コメント 5 件 ·
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
PolicyEngine/policyengine-us#10262 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
kellyjonbrazil/jc#788 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
`/search.json` ignores the documented `editions.sort` parameter対応中かも @PranavD2905 が今日担当しました。 オープンAffects: Server Good First Issue Module: Solr Needs: Triage Theme: Public APIs Theme: Search
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
internetarchive/openlibrary#13989 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug P2: misleading query
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
jeffknupp/association#478 ·