Bug 1468873 - Review Request: python-click-plugins - Click extension to enable registering CLI commands via setuptools entry-points
Summary: Review Request: python-click-plugins - Click extension to enable registering ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1468877
TreeView+ depends on / blocked
 
Reported: 2017-07-09 08:26 UTC by Elliott Sales de Andrade
Modified: 2017-08-31 14:54 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-08-31 14:54:35 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+


Attachments (Terms of Use)

Description Elliott Sales de Andrade 2017-07-09 08:26:54 UTC
Spec URL: https://copr-be.cloud.fedoraproject.org/results/qulogic/geopy/fedora-rawhide-x86_64/00577183-python-click-plugins/python-click-plugins.spec
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/qulogic/geopy/fedora-rawhide-x86_64/00577183-python-click-plugins/python-click-plugins-1.0.3-2.fc27.src.rpm
Description: This extension module for click allows registering external CLI commands via setuptools entry-points.

This is my first package in Fedora proper; I am seeking a sponsor.

Fedora Account System Username: qulogic

Comment 2 Zbigniew Jędrzejewski-Szmek 2017-08-12 19:13:20 UTC
I like the following trick:
%global _description \
An extension module for click to register external CLI commands via setuptools\
entry-points.

%description %_description
%description -n python2-%{srcname} %_description
%description -n python3-%{srcname} %_description

... but of course repeating the description text is OK.

Minor nitpick: the description line is very long, dnf list will look ugly.
What about: "Click extension to register CLI commands via setuptools"?

This is nice and straightforward:
+ package name is OK
+ license is acceptable for Fedora (MIT)
+ license is specified correctly
+ follows the modern python packaging template
+ builds and installs OK
+ no scriptlets needed
+ %check is present and passes
+ %license macro is used
+ %python_provide is used

rpmlint:
Only false positive ramblings about toadstools.

Package is APPROVED.

Comment 3 Gwyn Ciesla 2017-08-18 11:07:24 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/python-click-plugins

Comment 4 Fedora Update System 2017-08-19 06:47:18 UTC
python-click-plugins-1.0.3-3.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-cb75d0bea9

Comment 5 Fedora Update System 2017-08-22 18:07:05 UTC
python-click-plugins-1.0.3-3.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-cb75d0bea9

Comment 6 Fedora Update System 2017-08-31 14:54:35 UTC
python-click-plugins-1.0.3-3.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.