Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix annotations generated by py2pyi #613

Merged

Conversation

comhar
Copy link
Contributor

@comhar comhar commented Aug 23, 2024

Currently py2pyi generates fully qualified names (e.g. mod.submod.func) for annotations of user-defined types. These fully qualified names don't match the import statements in the node tree which can break downstream applications such as "jump to definition" in IDEs.

This PR resolves that issue by adding the correct import paths to the annotations.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@comhar
Copy link
Contributor Author

comhar commented Aug 23, 2024

@jph00

@jph00 jph00 added the bug Something isn't working label Aug 23, 2024
@jph00
Copy link
Contributor

jph00 commented Aug 23, 2024

Nice job!

@jph00 jph00 merged commit 40e73e7 into AnswerDotAI:master Aug 23, 2024
12 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants