chainlink-mix icon indicating copy to clipboard operation
chainlink-mix copied to clipboard

Add support for Type Hinting

Open igormorgado opened this issue 2 years ago • 5 comments

As now, Type hinting is strongly recommended by the whole Python community and is a huge step ahead while debugging, since that information can be used for static analysis.

The cost to add Type Hinting is minimal, requires small knowledge, and makes the code more robust, secure and readable.

Do you think that fits your project goals? If so, I can be a volunteer to this task.

igormorgado avatar Sep 26 '21 01:09 igormorgado

I think it does, to be honest. Type hinting is something I've been back and forth on actually. For bigger projects, it's basically essentail.

I'm trying to find the best balance though, there is a bit of a learning curve for newer engineers not used to type hinting. Is there some way we can account for this?

PatrickAlphaC avatar Sep 27 '21 18:09 PatrickAlphaC

Just considered this idea also. Is this an option to create a separate branch similar as it's done with javascript and typescript in some other smartcontractkit repos?

Sevamove avatar Apr 14 '22 15:04 Sevamove

@vsevdrob yes, that would be awesome!!

PatrickAlphaC avatar Apr 14 '22 21:04 PatrickAlphaC

Is anyone working actively on this issue? I'd like to contribute on this.

nitul1991 avatar Apr 24 '22 18:04 nitul1991

@nitul1991 Not sure... But it would be nice to see the first contribution on that theme!

Sevamove avatar Apr 24 '22 18:04 Sevamove