Modified CI to add black/isort/flake8
Modified also in adding coverage / removing pytest-cov (it's possible to debug with pytest-cov in pycharm but I always end up being frustrated by the user experience, see https://stackoverflow.com/a/40906766 )
the last commit I would prefer you double triple check, coverage indicated that
if user is None and group is None: was never False, which seems correct since we initialized it a few lines above with user, group = None, None
So I removed those lines and indented correctly, seems ok to me but maybe there was a reason you wrote this in the first place ?
Modified CI to add black/isort/flake8 Modified also in adding coverage / removing pytest-cov (it's possible to debug with pytest-cov in pycharm but I always end up being frustrated by the user experience, see https://stackoverflow.com/a/40906766 )
the last commit I would prefer you double triple check, coverage indicated that
if user is None and group is None:was never False, which seems correct since we initialized it a few lines above withuser, group = None, NoneSo I removed those lines and indented correctly, seems ok to me but maybe there was a reason you wrote this in the first place ?