https://bugzilla.redhat.com/show_bug.cgi?id=1458394 --- Comment #6 from Vít Ondruch <vondruch@xxxxxxxxxx> --- (In reply to mgansser@xxxxxxxx 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 -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
