Skip to content

Conversation

@robsdedude
Copy link
Member

With from __future__ import annotations in the code, Sphinx gets easily
confused. This applies to several areas like:

  • Type aliases
  • Things defined in if typing.TYPE_CHECKING blocks
  • Usage of the typing_extensions module

This PR aims to work around as many of those quirks as possible.

With `from __future__ import annotations` in the code, Sphinx gets easily
confused. This applies to several areas like:
 * Type aliases
 * Things defined in `if typing.TYPE_CHECKING` blocks
 * Usage of the `typing_extensions` module

This PR aims to work around as many of those quirks as possible.
Copy link
Contributor

@bigmontz bigmontz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✍🏼

Copy link
Contributor

@thelonelyvulpes thelonelyvulpes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of questions but otherwise looks good.

@robsdedude robsdedude merged commit 79747bb into neo4j:5.0 Sep 18, 2023
@robsdedude robsdedude deleted the docs-improve-types branch September 18, 2023 12:38
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.

3 participants