SG-29997 - FIRST PHASE Python2 removing

[eduardoChaucaGallegos]

• ensure_* functions replaced by function from sgutils.py.
• six types replaced by native

[julien-lang]

I see that you are adding the "ensure_" functions to a module but I don't see them being removed from the six module. Should not we do it?

[julien-lang]

Minor: the two functions don't use the same condition logic.

    if isinstance(s, str):
        return s

    elif isinstance(s, bytes):
        return s.decode(encoding, errors)
 
    raise TypeError(f"not expecting type '{type(s)}'")

[carlos-villavicencio-adsk]

LGTM

[mathbou]

Just a little reminder here, even if it's already merged; most of the work has already been done months ago

[eduardoChaucaGallegos]

Just a little reminder here, even if it's already merged; most of the work has already been done months ago

@mathbou Thanks for your collaboration, we have considered your changes for this PR. We did a new PR because we are trying to take less risk and doing this python2 removal in phases.