https://bugzilla.redhat.com/show_bug.cgi?id=1656628
Bug ID: 1656628 Summary: Review Request: python-readme-renderer - Safely render long_description/README files in Warehouse Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: rbarlow@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://bowlofeggs.fedorapeople.org/python-readme-renderer.spec SRPM URL: https://bowlofeggs.fedorapeople.org/python-readme-renderer-24.0-1.fc30.src.r... Description: Readme Renderer is a library that will safely render arbitrary README files into HTML. It is designed to be used in Warehouse to render the long_description for packages. It can handle Markdown, reStructuredText (.rst), and plain text. Fedora Account System Username: bowlofeggs
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
Randy Barlow rbarlow@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1551178
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1551178 [Bug 1551178] python-twine-1.11.0 is available
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
Jeremy Cline jeremy@jcline.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jeremy@jcline.org Flags| |fedora-review+
--- Comment #1 from Jeremy Cline jeremy@jcline.org --- I can't get fedora-review to work, but a manual inspection shows no problems.
Approved.
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
Robert-André Mauchin zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |zebob.m@gmail.com
--- Comment #2 from Robert-André Mauchin zebob.m@gmail.com --- Do you really need a docs subpackage for two files??
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
Robert-André Mauchin zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |POST Assignee|nobody@fedoraproject.org |jeremy@jcline.org
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
--- Comment #3 from Randy Barlow rbarlow@redhat.com --- https://pagure.io/releng/fedora-scm-requests/issue/9150
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
--- Comment #4 from Gwyn Ciesla limburgher@gmail.com --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-readme-renderer
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
Randy Barlow rbarlow@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |MODIFIED Fixed In Version| |python-readme-renderer-24.0 | |-1.fc30
--- Comment #5 from Randy Barlow rbarlow@redhat.com --- Waiting on https://src.fedoraproject.org/rpms/python-bleach/pull-request/2 and https://src.fedoraproject.org/rpms/python-bleach/pull-request/3 to bring this to f29 and f28.
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
--- Comment #6 from Randy Barlow rbarlow@redhat.com --- (In reply to Robert-André Mauchin from comment #2)
Do you really need a docs subpackage for two files??
No, but I also don't think it hurts.
https://bugzilla.redhat.com/show_bug.cgi?id=1656628
--- Comment #7 from Randy Barlow rbarlow@redhat.com --- Waiting on https://bodhi.fedoraproject.org/updates/FEDORA-2018-bf2cb9e51c and https://bodhi.fedoraproject.org/updates/FEDORA-2018-35faffe85f to go stable to build this for F29 and F28.
package-review@lists.fedoraproject.org