remove Python2 comptability code

[a-detiste]

No description provided.

[kingbuzzman]

@tristanlatr while removing mock would be a good add. This PR I say is ready to be merged

[a-detiste]

Most of my changes were picked up in other commits/ways. There's now here very little left.

[kingbuzzman]

Small commits are better than large commits 😉

This is great!

@bw2 / @tristanlatr can you authorize this PR to run CI. And if it passes... merge it? This PR is 2 years old 🤦‍♂️

[kingbuzzman]

@a-detiste can you resolve your conflicts?

[kingbuzzman]

@a-detiste thank you.
@bw2 gentle ping.

[kingbuzzman]

@a-detiste im so sorry, can you merge again? The merge of this PR really caused a lot of casualties 😅 -- it wasnt squashed and i kept syncing with master and fixing conflicts as new changes kept coming in.. and well it thinks i changed everything 😇

[a-detiste]

@kingbuzzman we are down to one line now.

In my eye the addition of toml dependency is a regression.

https://wiki.debian.org/Python/Backports

But that should be handled in another PR

[kingbuzzman]

we are down to one line now.

Oh right, It was me back in May... before I even saw your PR 😇

In my eye the addition of toml dependency is a regression.

Kinda.. not really, since this is what is used now...

But that should be handled in another PR

Agreed.

[bw2]

sorry for missing this earlier. It looks like the changes here have been merged via other PRs so am going to close this one.

[kingbuzzman]

It looks like the changes here have been merged via other PRs

This is not factual. There is still a single line that needs to be removed (this is master)

[kingbuzzman]

@bw2 may i suggest you authorize this PR to run the CI, if everything passes, it means mock is truly not needed, and thus this can be merged. Otherwise it can remain closed.

[a-detiste]

yes please