Probably needs some new typing tests too?

It's all pretty straightforward so I didn't think it would be necessary to add additional tests for this. I figured that as long as the current type-tests involving StringDType still pass, then it's fine. Mypy primer also shows no diff.

Testing this would also be rather difficult because it involves typing.Never. If you'd do something like assert_type(StringDType().na_object, typing.Never), then mypy doesn't analyze anything that comes after that statement, because it assumes that accessing na_object: typing.Never will raise an exception. It's possible to work around that by wrapping it in a function, but it'd be a lot of effort for a rather trivial annotation.

But if you want to see some test either way, I won't mind adding some 🤷🏻

But if you want to see some test either way, I won't mind adding some 🤷🏻

Eh, no worries.

Thanks Joren.