search

Uninett / zinolib

Python library for zino
Apache License 2.0
1 stars 3 forks source link

Add proper type annotations to config parsing modules #30

Open lunkwill42 opened 11 months ago

lunkwill42 commented 11 months ago 
          Why use docstrings when there are type annotations available?
    def update_from_args(self, args: argparse.Namespace):
        """Updates the configuration from command line arguments.

        arg-names not found in the config-object are ignored.
        """

_Originally posted by @lunkwill42 in https://github.com/Uninett/zinolib/pull/22#discussion_r1361781135_

@hmpf claims great haste in #22 and declines adding type annotations until the API is "more stable". So, I'm adding this issue to remind us to actually do this at some point.