PR#77: Remove readonly check when only getting plugin information - - Pagure.io

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

ipsilon

#77 Remove readonly check when only getting plugin information

Merged Feb 29, 2016 by puiterwijk. Opened Feb 29, 2016 by puiterwijk.

ticket-210 into master

Download 77.patch

no initial comment

Pull-Request has been rebased

Feb 29, 2016

Pull-Request has been updated

Feb 29, 2016

Pull-Request has been updated

Feb 29, 2016

rcritten commented Feb 29, 2016

I think more info is needed in either the commit message or the ticket (or both). I can't quite tell what this is supposed to be fixing.

I think the test change should probably move to the second patch.

Pull-Request has been rebased

Feb 29, 2016

Pull-Request has been updated

Feb 29, 2016

Pull-Request has been rebased

Feb 29, 2016

rcritten commented Feb 29, 2016

ACK

Tested with just the new test case and can clearly see the failure. Removing read-only test resolves issue.

puiterwijk commented Feb 29, 2016

Commit 7f76c760 fixes this pull-request

Pull-Request has been merged by puiterwijk@redhat.com

Feb 29, 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.