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



--- Comment #3 from Mamoru TASAKA <mtas...@fedoraproject.org> ---
Two comments

* License
  - As lib/liquid/module_ex.rb is under Ruby,
    the License tag should be "MIT and Ruby".

* Enabling tests
  - Would you actually execute test suite as much
    as possible, like below (for example)?

mkdir spy
touch spy/integration.rb
ruby -I"lib:.:test" -e 'Dir.glob("./test/**/*_test.rb").each{|f| require f
unless /context_unit_test/ =~ f }'

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to