-
Notifications
You must be signed in to change notification settings - Fork 45
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
Issue #424 passivation/activation and Annotated equality #430
Issue #424 passivation/activation and Annotated equality #430
Conversation
.../tests/full/context/passivating/dependency/builtin/BuiltinBeanPassivationDependencyTest.java
Show resolved
Hide resolved
Could you please rebase, or in some other way get rid of the 1st commit, which I think doesn't belong to this PR? |
Weird, but yes I'll see if I can get ride of the last commit |
9a1ec39
to
84ffcdd
Compare
Hopefully it's better. I extracted the method into the assert util class so it can be reused somewhere else if needed as your proposal was a bit more generic than what we needed |
LGTM, though you didn't necessarily have to remove the 2 asserts you added in the 1st commit, those are OK too. |
Added them back for completeness. |
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.
Looks good, thanks!
This small PR addresses the challenge of #424