[Bug 1271829] Review Request: nodejs-domutils - Utilites for working with htmlparser2's dom

bugzilla at redhat.com bugzilla at redhat.com
Tue Oct 20 21:52:17 UTC 2015


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

Troy Dawson <tdawson at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tdawson at redhat.com



--- Comment #1 from Troy Dawson <tdawson at redhat.com> ---
This package is missing the lib directory, which is critical, because the
index.js requires all the files in it.

Also, for the tests, can you put 
 %global enable_tests 0
at the top, so that when the tests do get fixed upstream, it will be simple to
get them going.

I will review it when those two things get done, because other than them, this
looks good.

-- 
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