-
Notifications
You must be signed in to change notification settings - Fork 46
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 JDT Java compiler entry for removed Java 7 support in ecj #213
Added JDT Java compiler entry for removed Java 7 support in ecj #213
Conversation
928f433
to
72e0b27
Compare
See screenshot here: #211 (comment) |
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.
Small nits.
I.e.
is always followed by a comma, i.e., i.e.,
.
The last sentence should probably mention javac
rather than just mention Java 21
, e.g., this is equivalent to Java 21's javac only supporting...
In any case, thanks for the rapid response. 🚀 ❤️
Sorry, I seem to have created more noise than necessary. 😢 |
72e0b27
to
bf7fc88
Compare
@iloveeclipse it would be good if you can take a look, or suggest someone from JDT who should take a look. I'm not sure if there is enough information in the N&N entry or if the wording is sufficient, to make the change clear to users. |
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.
From my point of view it looks okay.
Please also add a note that this applies also for the IDE, not only compiler. Thanks. I'm on the road today, so can't guarantee I can review anything. |
Added an entry for JDT, mentioning the removal of source, target and release Java 7 and below. See the respective JDT issue: eclipse-jdt/eclipse.jdt.core#2536 Signed-off-by: Simeon Andreev <[email protected]>
bf7fc88
to
383e84a
Compare
OK, I think we can merge this then. @merks would you merge this? |
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.
Thanks
<td class="title"><a href="#removed-support-for-java7-and-below">Removed support for souce, target and release Java 7 and below</a></td> | ||
<td class="content"> | ||
<!-- https://github.com/eclipse-jdt/eclipse.jdt.ui/pull/1367 --> | ||
<p>The following compiler options are now supported only for Java 8 and above: |
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.
This <p>
is unclosed and should be closed here.
Added an entry for JDT, mentioning the removal of
--source
,--target
and--release
Java 7 and below.See the respective JDT issue:
eclipse-jdt/eclipse.jdt.core#2536