[Bug 1298622] Review Request: rubygem-minitest5 - minitest provides a complete suite of testing facilities

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

 



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

Mamoru TASAKA <mtasaka@xxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+



--- Comment #3 from Mamoru TASAKA <mtasaka@xxxxxxxxxxxxxxxxx> ---
As this is basically EPEL branch request for existing rubygem-minitest master,
approving as it is.

-------------------------------------------------
    This package (rubygem-minitest5) is APPROVED
    by mtasaka
-------------------------------------------------

Some suggestions:
http://pkgs.fedoraproject.org/cgit/rpms/rubygem-minitest.git/commit/?id=409c32489ff9a9746976f73a7b315cdbc4d3d3cd

- Consider to update to 5.8.4
- Rakefile is not needed as this is alike "Makefile" on autotools
  system.
- We suggest removing test/ directory on runtime package:
 
https://fedoraproject.org/wiki/Packaging:Ruby?rd=Packaging/Ruby#Running_test_suites
"Do not ship tests"

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