danielgtaylor / danielgtaylor/python-betterproto

warnings package not imported for deprecated warning.

オープン
#239 コメント 1 件 リアクション 1 件 担当者 0 名 GitHub で見る
compiler-bug good first issue
主要言語
Python
スター
1.8k
フォーク
234
PR マージ指標
30日以内にマージされた PR はありません

説明

Hi there, thanks for the great project!

My environment:
betterproto: v2.0.0b3
python: v3.8.5
OS: Ubuntu 20.04

I use `deprecated` option in a message definition. betterproto generates a `warnings.warn` properly, but the `warnings` package is not imported, so it throws `NameError: name 'warnings' is not defined`. I manually add the line for now, but it'd be great if it can be fixed in the package.

Thanks!

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

betterproto v2.0.0b3 で非推奨のオプションを再現し、生成された Python 出力を調査します。生成されたコードが warnings パッケージをインポートし、NameError を発生させずに非推奨警告を出力すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
devtools
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。