None is not a valid value for ast.ImportFrom.level #92986

hauntsaninja · 2022-05-19T20:45:13Z

For example:

import ast
tree = ast.parse("from x import y")
tree.body[0].level = None
ast.unparse(tree)  # raises

This is very similar to https://bugs.python.org/issue46289 and #30467 cc @isidentical

Note the compiler does seem to allow it, but it's always an int in practice and documented as such

Discovered in typeshed.

The text was updated successfully, but these errors were encountered:

This doesn't happen naturally, but is allowed by the ASDL and compiler. We don't want to change ASDL for backward compatibility reasons (python#57645, python#92987)

hauntsaninja added the type-bug label May 19, 2022

hauntsaninja added a commit to hauntsaninja/cpython that referenced this issue May 19, 2022

pythongh-92986: Make level of ImportFrom not optional on ASDL

ffa0d95

AA-Turner added the stdlib label May 23, 2022

None is not a valid value for ast.ImportFrom.level #92986

None is not a valid value for ast.ImportFrom.level #92986

Comments

hauntsaninja commented May 19, 2022 • edited

hauntsaninja commented May 19, 2022 •

edited