-
-
Notifications
You must be signed in to change notification settings - Fork 952
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
fix #13655 - restore dot notation navigation #13869
fix #13655 - restore dot notation navigation #13869
Conversation
@jdaugherty why were all those |
That was my assumption, btw: this is still a draft pull request. I need to make sure the functional tests are passing before I can activate this. |
if that is the case, is going back to Java an option since this class is used so much? |
I'll look into it. |
def80d0
to
b92fc87
Compare
@@ -488,4 +489,172 @@ class NavigableMap implements Map<String, Object>, Cloneable { | |||
boolean equals(Object obj) { |
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.
@paulk-asert This class is tagged with both @EqualsAndHashCode
and it then implements it's own equals / hashcode. Do you know if the groovy transformation will have any effect?
I went ahead and marked this for review so we can restore this function for the milestone. I'll work on the java conversion separately from this PR and will leave the ticket open. |
No description provided.