sopel: drop support for Python 3.6 and require 3.7+

[Exirel]

Description

Tin. Python 3.6 died yesterday. I checked the warning for 3.6, they were written for Sopel 8, so we can either keep 3.6 or drop it now. Either way, this is not a big change, unlike Python 2.

I haven't modified the usage of OrderedDict, because there are slight difference between dict and OrderedDict in particular when considering equality.

I haven't check the "future" import either, hence the "draft" status for now.

Checklist

• I have read CONTRIBUTING.md
• I can and do license this contribution under the EFLv2
• No issues are reported by make qa (runs make quality and make test)
• I have tested the functionality of the things this change touches

[dgw]

Might as well take this opportunity to add 3.10 to CI, no?

[Exirel]

Test suite (3.6) Expected — Waiting for status to be reported

I've no idea what I missed here.

[dgw]

Test suite (3.6) Expected — Waiting for status to be reported

I've no idea what I missed here.

It's because that check is marked as required in the repo settings. Before merging I would just have to update the list of expected checks.

[dgw]

Suggestions from me should be the only changes needed to add Python 3.10 to CI and reflect its "officially supported" status in the package once built.

[Exirel]

So Github CI thinks it's "3.1" and not "3.10"... weird.

[Exirel]

Alright, now we can see that Python 3.10 is not supported by requests yet. See also: https://pyreadiness.org/3.10/

[dgw]

I see a lot of our transitive requirements in the "not supported" category, too—stuff we don't use directly, but that things we require depend on.

Let's abandon adding 3.10 in this patch and try again after the next requests release. (2.27 is supposed to be the last that will support Python 2.7, too, so that will be time to revisit our 7.x branch.)

[Exirel]

Weird, there is an isup test that fails in all three versions. I'm looking at potential cause, so here is the version installed:

• requests-2.27.1
• requests-mock~=1.9.3

I'll test locally to see if I can reproduce or not.

[Exirel]

I can confirm that with requests 2.26 the test pass, not with 2.27. This will need another PR to fix that.

[Exirel]

Following Python 3.7 documentation, I removed the generator_stop and added the annotation future imports.

[dgw]

One nit, for your INI-tweaking pleasure.

[Exirel]

Yup, fixed it and squashed it. Didn't change anything else.

[dgw]

I guess passing CI is our proof that trailing commas in flake8 config don't matter. 😁