tests drop macOS py38

[Borda]

What does this PR do?

Fixes #2039 (comment)

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

[codecov]

Codecov Report

Merging #2054 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #2054   +/-   ##
======================================
  Coverage      86%     86%           
======================================
  Files          75      75           
  Lines        4705    4705           
======================================
  Hits         4064    4064           
  Misses        641     641           

[SkafteNicki]

I will hopefully have time in next week to look more into what is causing this particular test to fail on Mac, so we can get the mac+python3.8 up and running again

[awaelchli]

Do we know which commit introduced it?
I suspect it is the #2029 but not sure. I would start backtracking it by resetting the head to before that PR and see when the tests start to hang.

[Borda]

Do we know which commit introduced it?
I suspect it is the #2029 but not sure. I would start backtracking it by resetting the head to before that PR and see when the tests start to hang.

Yes, it is... 8-)

[Borda]

@williamFalcon if we want to have skipped just a particular test I or someone else has to be running it locally, debuting it on CI would take days as it is incremental work and each iteration takes about 20min so I propose to drop the macOS with py3.8 completely, for now, to unblock waiting all PRs and in meantime @SkafteNicki will find the issue..