https://bugzilla.redhat.com/show_bug.cgi?id=1231564 Parag AN(पराग) <panemade@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #7 from Parag AN(पराग) <panemade@xxxxxxxxx> --- (In reply to Piotr Popieluch from comment #6) > Thank you for review. > > elasticdumps intended usage is as a tool (for exporting/importing databases) > not as a library. Therefore I think "elasticdump" suits better than > "nodejs-elasticdump". > > In the nodejs guidelines: > - Application packages that mainly provide tools (as opposed to libraries) > that happen to be written for Node.js must follow the general naming > guidelines instead. umm.. Okay. I thought to give precedence to library naming rule, but I think we can also accept this with above tools rule. APPROVED. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review