PR#213: Fix specifying a configuration file in the runserver - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

fedora-hubs

#213 Fix specifying a configuration file in the runserver

Merged Jul 14, 2016 by pingou. Opened Jul 13, 2016 by pingou.

pingou/fedora-hubs fix_runserver into develop

Download 213.patch

no initial comment

rebased

Jul 13, 2016

sayanchowdhury commented Jul 13, 2016

os.path.join needed?

sayanchowdhury commented Jul 13, 2016

Looks good :thumbsup:

pingou commented Jul 13, 2016

I'm going to wait for @atelic to confirm that this works before merging, thanks for the review :)

atelic commented Jul 13, 2016

Confirmed that it works for me. :thumbsup:

atelic commented Jul 13, 2016

You will want to change the help text since that format won't work anymore

Edited Jul 13, 2016 by atelic

1 new commit added

Adjust the help message for the config CLI argument

Jul 13, 2016

pingou commented Jul 13, 2016

@atelic, is this better?

atelic commented Jul 13, 2016

alloing

Allowing? :)

1 new commit added

Typi, typo

Jul 13, 2016

atelic commented Jul 13, 2016

LGTM

Pull-Request has been merged by pingou

Jul 14, 2016

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

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

© Red Hat, Inc. and others.