I like the idea and the change, though I suspect a few tests need to be adjusted for this :)

Yes, I also agree to have tests, but my instance is running without problem as of now.

Looks like there are three tests failing:

tests/test_pagure_flask_ui_groups.py:201: AssertionError
tests/test_pagure_flask_ui_app.py:2649: AssertionError
tests/test_style.py:105: AssertionError

There is another one at tests/test_pagure_flask_ui_login.py:1136: but that seems weird

I ran python -m black -l 79 --check --diff --exclude "/(\.eggs|\.git|\.hg|\.mypy_cache|\.nox|\.tox|\.venv|_build|buck-out|build|dist)/" ../pagure tests manually, then got

would reformat ../pagure/runserver.py
Oh no! 💥 💔 💥
91 files would be reformatted, 340 files would be left unchanged.

It seems many files hit.

Check the version of black you're using, we're still using black == 19.10b0; only

I also run black simply using black -l 79 tests/test_*.py pagure

Also, please use git rebase instead of git merge, we try to keep the history linear and avoid merge commits as much as possible :)

I ran black -l 79 tests/test_*.py pagure, then I got:

pagure@lb:~/pagure$ black -l 79 tests/test_*.py pagure
reformatted /srv/pagure/pagure/pagure/hooks/irc.py
reformatted /srv/pagure/pagure/pagure/forms.py
reformatted /srv/pagure/pagure/pagure/login_forms.py
All done! ✨ 🍰 ✨
3 files reformatted, 194 files left unchanged.

with no error.

Did you delete your fork?

Yes. please close this PR.