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

Douglas Schilling Landgraf <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #4 from Douglas Schilling Landgraf <[email protected]> ---
Hi,

(In reply to Yaniv Bronhaim from comment #3)
> As part of 1.92-2 you should modify the README file. And if you already
> modify it you should also:
> 1. split the README and AUTHORS, it's more common. 
> 2. remove the .txt suffix (also from LICENSE.txt and ChangeLog.txt)
> 3. Make the spec URL fits the README URL.

Yaniv thanks for your informal review and suggestions. 
Tomasz, all suggestions shared by Yaniv are not blockers but you can
suggest/handle them with upstream developers, for example, opening a
ticket/bugzilla and update your package again when they release a new version.
My next comment in this bugzilla will be my review.

Cheers
Douglas

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=exlEqWFu6x&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to