We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
We would like all 3 examples to return true. === should continue to return false.
===
[11] pry(main)> id = Valkyrie::ID.new('abc') => #<Valkyrie::ID:0x007fdee25d4760 @id="abc"> [12] pry(main)> id.eql?('abc') => false [13] pry(main)> id == 'abc' => false [14] pry(main)> id == Valkyrie::ID.new('abc') => true
The text was updated successfully, but these errors were encountered:
I didn't do this on purpose for some reason and now I don't remember why, so okay.
Sorry, something went wrong.
Related to #765
Happy to implement this, if there's broad agreement it's desirable.
It's probably a major version, once in.
no-reply
No branches or pull requests
We would like all 3 examples to return true.
===
should continue to return false.The text was updated successfully, but these errors were encountered: