https://bugzilla.redhat.com/show_bug.cgi?id=1136385 --- Comment #3 from Josef Stribny <jstribny@xxxxxxxxxx> --- 1, Fix spelling: "Rspec-2 doubles (mocks and stubs)" should be "RSpec 2 doubles (mocks and stubs)" "rspec-mocks provides a test-double framework for rspec including support for method stubs, fakes, and message expectations." should be "rspec-mocks provides a test-double framework for RSpec including support for method stubs, fakes, and message expectations." 2, Why do you need two macros for bootstrapping? Isn't need_bootstrap enough? 3, Use single spaces rather than tabs: "%doc %{gem_instdir}/*.md" should be rather "%doc %{gem_instdir}/*.md" This is my personal opinion; using tabs looks like it's badly formatted. -- 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