This function doesn't fetch data from Pagure, it creates a Pagure object, which holds the information about Pagure instance. But fetching of data happens only when you use the methods of this object.

True

Usually when you name or describe the function, try to avoid too much grammar in the name. If it is a verb - let it be the infinitive.

For example:

function filter_pull_requests

description Filter pull requests according to additional parameters

And so on.

It saves space on the screen, time for typing, and sounds more direct. And also later on you don't need to remember the exact wording, you always go for infinitive form when search for smth

:thumbsup:
I'll amend it.

Since we set pagure instance as a global object, we don't need to pass it as a parameter, simply use it inside the function

Maybe use capitalized P here as in P = libpagure.Pagure(pagure_token=api_token, pagure_repository=project) or PAGURE?

To clarify that we are going to use it as a global object not a local variable

ups, I forgot to remove it, that was only for some testing purposes :)

Maybe use capitalized P here as in P = libpagure.Pagure(pagure_token=api_token, pagure_repository=project) or PAGURE?
To clarify that we are going to use it as a global object not a local variable

ok, I'll fix it!