Skip to content

Simplify part of, type and cleanup py2d.py#2385

Open
Avasam wants to merge 13 commits intomhammond:mainfrom
Avasam:cleanup-py2d
Open

Simplify part of, type and cleanup py2d.py#2385
Avasam wants to merge 13 commits intomhammond:mainfrom
Avasam:cleanup-py2d

Conversation

@Avasam
Copy link
Collaborator

@Avasam Avasam commented Oct 6, 2024

Split off from #2384 (which only aims at updating online docs with 307).
Helps towards #2432

This PR:

  • Adds type annotations to py2d.py for development and IntelliSense convenience
  • Does the DocInfo initialization work in the DocInfo.__init__ method
  • Creates a subclass of DocInfo to do the same for argument infos generated by the BuildArgInfos method
    • Simplifies usages of should_build_function
  • Simplify print calls as sys.stdout is already the default when file=None

Edit: I can simplify this into smaller chunks for ease of review.

@Avasam Avasam requested a review from mhammond October 18, 2024 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant