Reenable Windows tests

[Dreamsorcerer]

No description provided.

[codecov]

Codecov Report

Merging #329 (07439a0) into master (88b7477) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #329   +/-   ##
=======================================
  Coverage   82.37%   82.37%           
=======================================
  Files          12       12           
  Lines         681      681           
  Branches       80       80           
=======================================
  Hits          561      561           
  Misses        102      102           
  Partials       18       18           

Flag	Coverage Δ
unit	82.37% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88b7477...07439a0. Read the comment docs.

[Dreamsorcerer]

@samuelcolvin Have you got any time to check out this error? Seems grablib fails with a UnicodeEncodeError on Windows (although the exception itself seems to come from click).

[Dreamsorcerer]

@samuelcolvin Also, do you have admin access? I'd like pyup removed and the required status set on the 3.9 tests (otherwise Dependabot will auto-merge broken dependencies).

[samuelcolvin]

Thanks, I'll try to look next week.

[samuelcolvin]

@Dreamsorcerer I've made you an admin. I'll look at grablib now.

[samuelcolvin]

@Dreamsorcerer please please can you use informative names for branches in github.

Couldn't be less helpful

[samuelcolvin]

as explained here windows tests were passing when they shouldn't be.

[Dreamsorcerer]

Seems the problem is only deprecation warnings internal to python (which is a result of a change I got merged into the last 3.9 release). Looks like everything is running now after your previous change.

Though curious the warnings didn't fail the tests for ubuntu/mac.

[Dreamsorcerer]

Also, I don't see any setting for pyup. Is it managed at https://pyup.io/? Would be nice to disable it, as dependabot should be working and auto-merging.

[samuelcolvin]

I deleted the webhooks.

[Dreamsorcerer]

I deleted the webhooks.

Still seem to be coming in for some reason:
#337