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 involvingStringDType still pass, then it's fine. Mypy primer also shows no diff.

Testing this would also be rather difficult because it involvestyping.Never. If you'd do something likeassert_type(StringDType().na_object, typing.Never), then mypy doesn't analyze anything that comes after that statement, because it assumes that accessingna_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.