[Bug 1656628] New: Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

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: [e-mail]
          Reporter: [e-mail]
        QA Contact: [e-mail]
                CC: [e-mail]

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

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

Randy Barlow  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

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

Jeremy Cline  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
              Flags|                            |fedora-review+

--- Comment #1 from Jeremy Cline  ---
I can't get fedora-review to work, but a manual inspection shows no problems.

Approved.

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

Robert-André Mauchin  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from Robert-André Mauchin  ---
Do you really need a docs subpackage for two files??

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

Robert-André Mauchin  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |POST
           Assignee|[e-mail]    |[e-mail]

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

--- Comment #3 from Randy Barlow  ---
https://pagure.io/releng/fedora-scm-requests/issue/9150

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

--- Comment #4 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-readme-renderer

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

Randy Barlow  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|POST                        |MODIFIED
   Fixed In Version|                            |python-readme-renderer-24.0
                   |                            |-1.fc30

--- Comment #5 from Randy Barlow  ---
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.

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

--- Comment #6 from Randy Barlow  ---
(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.

[Bug 1656628] Review Request: python-readme-renderer - Safely render long_description/ README files in Warehouse

https://bugzilla.redhat.com/show_bug.cgi?id=1656628

--- Comment #7 from Randy Barlow  ---
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.