https://bugzilla.redhat.com/show_bug.cgi?id=972719 Douglas Schilling Landgraf <dougsland@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED --- Comment #4 from Douglas Schilling Landgraf <dougsland@xxxxxxxxxx> --- 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 package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review