Closed clbarnes closed 2 years ago
I briefly pushed the branch to upstream which triggers the tests to actually run.
Thanks! MacOS fails because the image doesn't have mercurial in it by default. Other than that, seems to be passing on my side!
Before un-drafting this, I'll remove the appveyor and travis configs and add py39.
Just so you're aware of it: I now need to approve and run every time to actually run.
Could you also give it a name at the top level? That way we don't see the the filename but rather something nice. You can add name: CI
at the very top.
Don't worry about approving it so it can run here, I'm checking on my own branch and will un-draft when it's passing there. Working on the last failures!
Green on my end. py39's argparse apparently formats differently to others', so I had to normalise the whitespace when doing string containment checks on those messages.
Perhaps you can squash the commits and leave out the Python 3.9 changes? Then I'd have no problem merging it but I don't have time to look at the 3.9 changes that closely.
Rebased, updated the readme buttons, noted it in the changelog, and squashed. It's weird that appveyor is still doing anything; the repo doesn't have any mentions of it outside of the changelog. Maybe the repo is still registered on the appveyor side? It looks like the failure is that it can't find the config file, which is good at least.
@florisla this has been open for a long time. Any reason we can't merge this?
Much appreciated @clbarnes, this is great work!
See #227