#3855 Adding 'list-groups' function to pagure-admin
Merged by pingou. Opened by pingou.
admin_lists_groups  into  master

Download 3855.patch

Merges https://pagure.io/pagure/pull-request/3703

1 new commit added

  • Black fix

Thanks :)

rebased onto 80c1b337c33f417682dbffb9c23a8f5431573aed

I see one failing test locally, I'll fix it before merging

This is the change I've had to add to make the tests pass:

--- a/ tests/test_pagure_admin.py       
+++ b/ tests/test_pagure_admin.py       
@@ -20,10 +20,10 @@ import shutil  # noqa
 import subprocess  # noqa
 import sys  # noqa
 import unittest  # noqa
-from StringIO import StringIO  # noqa
 import munch  # noqa
 from mock import patch, MagicMock  # noqa
+from six import StringIO  # noqa

rebased onto 7f32810d282abd71baf28182392d346fa38bc978

Looks like Jenkins is back, let's wait for it :)

rebased onto 0096c93bfec4a0d677d9a7ad839988925beb3cc7

Pull-Request has been merged by pingou

Metadata