pagure

#3138 Fix default_hook by moving config env to top
Merged by pingou. Opened by puiterwijk.
puiterwijk/pagure hookfix  into  master

Download 3138.patch

Signed-off-by: Patrick Uiterwijk puiterwijk@redhat.com

LGTM. :thumbsup:

It shouldn't matter since the config is loaded after the environment variable is set, but looks fine regardless

rebased onto e5f4ce61a43e0718ae64265b67f951ca24a7efa3

Pull-Request has been merged by pingou

I think this causes a new flake8 error:

/code/pagure/hooks/files/default_hook.py:18:1: E402 module level import not at top of file

Obvious fix is to add another # noqa: E402...
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.