pagure

#3703 Adding 'list-groups' function to pagure-admin
Merged by pingou. Opened by arrfab.
arrfab/pagure pagure-admin-list-groups  into  master

Download 3703.patch

This commit add the "list-groups" function to pagure-admin cli tool.
Currently pagure-admin has a way to add (new-group) groups to pagure, but nothing to list those.
This can be useful for sysadmin writing tools around pagure-admin

@arrfab It looks like your code fails the code style of the project.

Can you please blacken the code using python3-black?

We should also add some tests for this

@ngompa I don't even understand what that means :) ... also it's still only python2 in the .el7 land, version on which I'm testing and (soon ?) deploying this .. can you elaborate ?

@pingou : sure : do you have docs/pointer for this ?

The existing tests for the cli are in tests/test_pagure_admin.py, I don't think we have much doc on how to write the tests themselves :s

@arrfab Pagure code follows a rigid code style imposed by a tool called black, which is available in Fedora as the python3-black package.

@ngompa I can have a look but myself not a coder/developer and that package doesn't exist for .el7 either

Commit 0096c93b fixes this pull-request

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.