Change PartitionKeyType from List to Sequence?

[trevorcreech]

Describe the bug
First off, thanks for all the typing you all have added!

Our partition_keys are always strings so we'd like our internal methods to have type Union[str, list[str], None], but that's not compatible with the PartitionKeyType in the SDK because lists are mutable

GitHub issue with explanation of why list[str] isn't compatible with e.g. list[str | int]

Assuming these functions never modify the partition_keys that are passed in, so could we change that type from List to Sequence?

To Reproduce
mypy playground: List failure case
mypy playground: Sequence success case

Expected behavior
Users of the sdk can restrict their partition key types to a stricter subset

[annatisch]

This is a great catch @trevorcreech - thank you for the report!
I believe your assumption is correct - @simorenoh are you able to confirm whether the partitionkey value is ever modified prior to the request being sent? If not, I think this should be a good change to make. I can update my existing paritionkey typing PR

[annatisch]

Ping @bambriz and @simorenoh on this one.

[bambriz]

@annatisch I think it would be a goof change, there shouldn't be any changes to the partition key values as it gets sent. It also wouldn't matter if the values are passed in a list or a tuple (in regard to subpartitioning pk values) as it will be the same for the json request headers.

[annatisch]

Done :)
Thanks @trevorcreech! This will go out in the next release.