added return types for getter methods

[nikbott]

Since the methods may not return the intended type, they're wrapped in an Optional. Although I think all methods should have explicit return types, that should be further analyzed.

[Kalebu]

Hi @nikbott

Is it this best practice when specifying return types than importing Optional from typing ?

from typing import Dict, Any, List, Optional

def my_method(self) -> Optional[Dict[Any, Any]]
  """
  My method does a lot
  """
  return {}

[nikbott]

According to the library documentation, Optional is simply a wrapper for a Union[X, None], which can be written as X | None. So both syntaxes should be valid and equivalent, and a matter of preference.

[Kalebu]

According to the library documentation, Optional is simply a wrapper for a Union[X, None], which can be written as X | None. So both syntaxes should be valid and equivalent, and a matter of preference.

Cool, Thanks for the clarification and your contribution

Merging this