-
-
Notifications
You must be signed in to change notification settings - Fork 388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove support for obsolete Prime.new #63
Conversation
it "raises a ArgumentError when is called with some arguments" do | ||
lambda { Prime.new(1) }.should raise_error(ArgumentError) | ||
lambda { Prime.instance(1) }.should raise_error(ArgumentError) | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should be removed and merged with instance_spec.rb.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the cases in new_spec are covered in instance_spec.rb, so we can remove the file without merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect then! Can you remove it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah! I will commit in sometime
Looks good! Sorry for replying late. |
Thank you for the review! Sure, I make the changes. |
Thank you for your contribution, it's much appreciated! |
remove support for obsolete Prime.new
Prime.new was removed in 2.3.0 ruby/ruby#891 ruby/spec#63
Prime.new was removed in 2.3.0 ruby/ruby#891 ruby/spec#63
deprecate Prime.new