[Bug 1228146] Review Request: lettuce - Scalable Java Redis client

bugzilla at redhat.com bugzilla at redhat.com
Thu Jun 18 06:33:01 UTC 2015


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

jiri vanek <jvanek at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-review+



--- Comment #3 from jiri vanek <jvanek at redhat.com> ---
missed one line:
[x]: Package uses nothing in %doc for runtime.

Hello Gil!

Form my side package is o to go, unless you wont follow Mark's hint from #1.

>From minor nits you may wont to improve:
lettuce.noarch: W: spelling-error %description -l en_US asyncronous ->
asynchronous
I think  in this case this mostly wrong check is right :)

Also may you enable check, or write an comment to spec why it is disabled? (so
why -f is needed )
I have not tried, but it whole looks so simple, that -f seems not necessary.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component


More information about the package-review mailing list