Closed hardingnj closed 4 years ago
Hello @hardingnj! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Also @hardingnj would be great if you could add an entry in the release notes (docs/release.rst) crediting yourself and @olavurmortensen.
All should be done! Always happy to relearn how to git rebase
properly :)
Cool, nice to see this done :)
Thanks @olavurmortensen and @hardingnj :-)
Supersedes #300
Thanks for your work on this @olavurmortensen. Apologies for the delay in review.