python-apps icon indicating copy to clipboard operation
python-apps copied to clipboard

Typehints for the Shuffle Apps

Open 00willo opened this issue 3 years ago • 2 comments

It may have already been discussed, but any interest in adding typehints to some of the apps?

It's not necessary for all of them, and can be optional. I can see some advantages to doing it. I feel like it'd assist in readability of the code at a minimum and assist help out the IDEs in being able to more accurately autocomplete code as you're typing.

00willo avatar Mar 26 '22 06:03 00willo

An additional advantage of adding typehints would make it easier for new or potential contributors to reason about smaller parts of the codebase.

00willo avatar Mar 26 '22 08:03 00willo

This makes sense from a stability standpoint short-term, but I think it loses its benefits long-term.

I've always thought types shouldn't be a thing users nor developers should have to care about. The goal is instead to get to a point where Shuffle "understands" what the type should be based on past behavior. This is e.g. why I despise Typescript personally, as it takes away all joy from development

Types suck, and make everything way more tedious for everyone

frikky avatar Mar 27 '22 15:03 frikky