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



--- Comment #6 from Vít Ondruch <[email protected]> ---
(In reply to [email protected] from comment #5)
> (In reply to Vít Ondruch from comment #4)
> > * Please describe patches
> >   - You carry aroun nuvoplaruntie-wscript patch, but what is it purpose?
> > Where
> >     is it coming from? Is it upstreamed or taken from upstream? While you
> >     probably know, it would be helpful to others to attache 2 line comment
> >     above the patch to explain the purpose and link the sources. Thx.
> 
> add description 

That is perfect. Thx.

> > * description
> >   - I'd keep just the first paragraph of the description (i.e. "Nuvola Apps
> >     is a runtime for semi-sandboxed web apps ...").
> > 
> shorten the description as you mentioned

I would remove the first two lines as well ...

> thanks for your review

YAW

-- 
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
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to