Skip to content
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

Merged

Conversation

trancexpress
Copy link
Contributor

@trancexpress trancexpress commented Aug 26, 2024

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

@trancexpress
Copy link
Contributor Author

See screenshot here: #211 (comment)

news/4.33/jdt.html Outdated Show resolved Hide resolved
Copy link
Contributor

@merks merks left a 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. 🚀 ❤️

news/4.33/jdt.html Outdated Show resolved Hide resolved
@merks
Copy link
Contributor

merks commented Aug 26, 2024

Sorry, I seem to have created more noise than necessary. 😢

@trancexpress
Copy link
Contributor Author

@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.

Copy link
Contributor

@merks merks left a 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.

@iloveeclipse
Copy link
Member

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]>
@trancexpress
Copy link
Contributor Author

OK, I think we can merge this then. @merks would you merge this?

Copy link
Member

@iloveeclipse iloveeclipse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@iloveeclipse iloveeclipse merged commit 8d734e8 into eclipse-platform:master Aug 26, 2024
1 check passed
<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:
Copy link
Contributor

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.

merks added a commit to merks/www.eclipse.org-eclipse that referenced this pull request Aug 26, 2024
@merks merks mentioned this pull request Aug 26, 2024
merks added a commit that referenced this pull request Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants