Spec URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-rspec-expectations/rubygem-rspec-expectations.spec SRPM URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-rspec-expectations/rubygem-rspec-expectations-2.0.1-1.fc.src.rpm Description: rspec-expectations adds `should` and `should_not` to every object and includes RSpec::Matchers, a library of standard matchers.
Updated. http://mtasaka.fedorapeople.org/Review_request/rubygem-rspec-expectations/rubygem-rspec-expectations.spec http://mtasaka.fedorapeople.org/Review_request/rubygem-rspec-expectations/rubygem-rspec-expectations-2.5.0-1.fc.src.rpm * Thu Feb 17 2011 Mamoru Tasaka <mtasaka.u-tokyo.ac.jp> - 2.5.0-1 - 2.5.0
I am taking this review.
* Unnecessary prep section - The "gem install" could be moved into install section. This is not MUST, however it would simplify the spec file a bit. * Test suite - Some specs are not executed (there are 4 specs missing). The test suite should be executed by following command preferably: ruby -rubygems -Ilib/ -S rspec spec - What about executing Cucumber test suite? But there is probably missing packaged Aruba gem :/ * Koji: - Builds fine on Koji http://koji.fedoraproject.org/koji/taskinfo?taskID=2859873 Neither one of the above is showstopper, so the package is APPROVED.
Thank you for review and comments. I will refrect them when I import this package into Fedora git. New Package SCM Request ======================= Package Name: rubygem-rspec-expectations Short Description: Rspec-2 expectations (should and matchers) Owners: mtasaka Branches: f15 f14 f13 InitialCC:
Git done (by process-git-requests).
Rebuilt for all branches, submitted push requests for F-15/14/13, closing. Thank you for the review and git procedure.
Package Change Request ====================== Package Name: rubygem-rspec-expectations New Branches: el6 Owners: bkearney vondruch InitialCC: