https://bugzilla.redhat.com/show_bug.cgi?id=1271829 Troy Dawson <tdawson@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tdawson@xxxxxxxxxx --- Comment #1 from Troy Dawson <tdawson@xxxxxxxxxx> --- 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 _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review