[Bug 1310375] Review Request: nodejs-qunitjs - An easy-to-use JavaScript Unit Testing framework

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #2 from Tom Hughes <tom@xxxxxxxxxx> ---
There is ASL 2.0 licensed code in src/diff.js so the license tag will need to
reflect that. Not sure if we should also add a bundled() provide for that code?

Do we need to ship qunit.css? Not sure what it's for as the js never seems to
touch it - presumably used if you run a qunit suite in a browser? If we do ship
it then if should likely be in /usr/share anyway.

Maybe a good idea to remove the generated files in qunit in %setup to make sure
we are generating new ones.

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




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]