Drop EOL Python 3.6

[ofek]

Discussion: #2136

Also, now uvicorn can be bumped (was pinned to fix Python 3.6 in #1807)

[Kludex]

Only @tomchristie can run the pipeline, right?

[florimondmanca]

@Kludex Shouldn't be the case, not sure why the pipeline didn't get triggered again after the merge. 🤔

[florimondmanca]

Here we go… I undid the merge and rebased. Not sure what happened. @ofek You might need to reset your py branch if you need to access it again.

[florimondmanca]

CI fails, need to upgrade mkdocs due to mkdocs/mkdocs#2799

[Kludex]

@Kludex Shouldn't be the case, not sure why the pipeline didn't get triggered again after the merge. 🤔

I think this happens because the pipeline listens to master's CI configuration file. 🤔

[Kludex]

Discussion: #2136

Also, now uvicorn can be bumped (was pinned to fix Python 3.6 in #1807)

That's misleading. uvicorn was not pinned because of 3.6. But because of another issue: encode/uvicorn#1161

But that's not a blocker for this PR.

[ofek]

black needs upgrading psf/black#2966

[ofek]

@tomchristie Would you mind triggering the CI with that button below?

[tomchristie]

@ofek - I'm not sure what you mean? The CI has run okay for this.

[Kludex]

I pressed the button 😅

[adriangb]

The changes look good to me. Approved on my end.

[florimondmanca]

Approving based on my line of reasoning here #2136 (reply in thread)

[florimondmanca]

(Is it new that PRs from first-time contributors must be approved to run each time? Is that a new GitHub thing? Configurable perhaps?)

[adriangb]

It's been a thing for a while. I believe it's to stop spam bots from mining crypto and stuff like that. A common trick for a well known contributor like @ofek is for them to just a no-brainer PR (like taking the removal of trailing whitespaces they're doing in this PR) which one of us can instantly approve and merge, making them no longer a first time contributor -> they can run CI on a more complicated PR like this one

[tomchristie]

y'all go for it, team.

[Kludex]

(Is it new that PRs from first-time contributors must be approved to run each time? Is that a new GitHub thing? Configurable perhaps?)

Not configurable yet.