Add typing and required imports

[tekktrik]

No description provided.

[tekktrik]

Addressed issues from pre-commit

[FoamyGuy]

Thanks for working on this @tekktrik

I think we want to put the imports inside of a try/except block like this:

try:
    # Used only for typing
    import typing
    from busio import I2C
    from time import struct_time
except ImportError:
    pass

This way the imports will not happen when it runs on the microcontroller which will end up saving some RAM.

[tekktrik]

That makes sense, so anything not functionally necessary (just for typing) won't actually get called because of the ImportError on typing? I'm doing a few other repos, so just want to make sure I understand correctly. Thanks for the feedback!

[FoamyGuy]

That makes sense, so anything not functionally necessary (just for typing) won't actually get called because of the ImportError on typing?

Yep that is correct. Importing anything from typing will raise the exception on microcontroller since there is no built-in typing module in CircuitPython.

[tekktrik]

There's another library I'm working on that needs typing that doesn't use anything from the typing module, how would I handle that, since adding it makes the pylint fail? Is that still okay?

See adafruit/Adafruit_CircuitPython_74HC595#19 for reference

[tekktrik]

Made edits per review

[FoamyGuy]

Looks good to me.