Skip to content

chore: Adds typing annotations to Coriolis - #542

Draft
claudiubelu wants to merge 1 commit into
cloudbase:mainfrom
claudiubelu:righttyper
Draft

claudiubelu wants to merge 1 commit into
cloudbase:mainfrom
claudiubelu:righttyper

Conversation

@claudiubelu

Copy link
Copy Markdown
Member

Adds argument and return type annotations to Coriolis, inferred from runtime types, by using RightTyper and running the integration tests.

Methodology:

  • collection: run the integration tests under RightTyper.
  • emission: righttyper process --python 3.10 --type-depth-limit=1
  • normalisation: self, cls, *args, **kwargs left without annotations. Container types are kept shallow. Removed -> None from __init__.

Adds argument and return type annotations to Coriolis, inferred from
runtime types, by using RightTyper and running the integration tests.

Methodology:
- collection: run the integration tests under RightTyper.
- emission: `righttyper process --python 3.10 --type-depth-limit=1`
- normalisation: `self`, `cls`, `*args`, `**kwargs` left without
  annotations. Container types are kept shallow. Removed `-> None` from
  `__init__`.
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