-
Notifications
You must be signed in to change notification settings - Fork 529
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
Added association not soft destroyed validator #389
Added association not soft destroyed validator #389
Conversation
Why the merge commit in e3eeb53? |
Looks like 51f9541 occurred in #385, yet @jhawthorn hasn't rebased. So I'll take yours :) |
@BenMorganIO because I first edited the README directly through GitHub, and then fixed the test warnings but forgot to pull first ;) |
test/paranoia_test.rb
Outdated
validates :parent_model, association_not_soft_destroyed: true | ||
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.
extra new line here
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.
fixed
@edwardmp awesome. Could you add this to the Changelog using the same format as in 2.2.1 and see if you can remove the merge commit? It's fine if you can't, they're tricky things merge commits. Thanks for all your work on getting this guy in!!! |
e3eeb53
to
fc16137
Compare
@BenMorganIO all fixed! |
5dd1ea1
to
fbd117f
Compare
fbd117f
to
d366099
Compare
Thanks! |
See #380.
While we're at it I also included some changes so the following warning doesn't show up anymore:
Use assert_nil if expecting nil from [..]. This will fail in MT6.