-
Notifications
You must be signed in to change notification settings - Fork 264
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
Drop compat code for java 8 #2155
base: master
Are you sure you want to change the base?
Conversation
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.
Let's get this in now, we could have gotten rid of this a while ago already
There is actually a catch, as c848cfe shows. |
I'm interested in your opinion on |
This is most consistent with Java API, it adds `obj.getClass.getPrettySimpleName` to the existing `obj.getClass.get*Name`. Instead we could go with a nicer `obj.prettyClassName`.
de8bc92
to
e43bcdd
Compare
Since this isn't trivial after all, I think it should wait for after the release. |
Foo.Bar -> "Bar", | ||
Foo.Baz(42) -> "Baz" | ||
).foreach { case (o, ref) => assert(o.getClass.getPrettySimpleName == ref) | ||
} |
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.
nit: indent is weird
It has been a long time and officially supported version is Java 11.
This can wait after the upcoming release though.