sphinx-contrib / documentedlist

DocumentedList Sphinx Extension
BSD 2-Clause "Simplified" License
4 stars 3 forks source link

drop six usage #5

Open pgajdos opened 1 year ago

pgajdos commented 1 year ago

Could you please consider to drop six usage, python 2 is over.

chintal commented 1 year ago

I have no serious objection to dropping six usage. The only thing we use from six is text_type, and changing it to use str alone would break the relatively narrow (?) cross section of code using unicode alongside 2.x.

That said, I'm curious what the upside is. There is a potential downside to making the change, however small - documentation is a pain in of itself, and having documentation tools force revisiting old code without any accompanying benefits feels like bad form to me.

As it stands we do maintain compatibility with old code which may be using this extension while not breaking anything with the new stuff.