danielgtaylor / danielgtaylor/python-betterproto

warnings package not imported for deprecated warning.

Open
#239 1 comment 1 reaction 0 assignees View on GitHub
compiler-bug good first issue
Dominant language
Python
Stars
1.8k
Forks
234
PR merge metrics
No merged PRs in 30d

Description

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!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.