Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
IEP-1792: Align execution environments to java 21 by sigmaaa · Pull Request #1488 · espressif/idf-eclipse-plugin · GitHub
Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal
, 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' IEP-1792: Align execution environments to java 21 by sigmaaa · Pull Request #1488 · espressif/idf-eclipse-plugin · GitHub
Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal
, 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' IEP-1792: Align execution environments to java 21 by sigmaaa · Pull Request #1488 · espressif/idf-eclipse-plugin · GitHub
Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal
, 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ' IEP-1792: Align execution environments to java 21 by sigmaaa · Pull Request #1488 · espressif/idf-eclipse-plugin · GitHub
Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal
, 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' IEP-1792: Align execution environments to java 21 by sigmaaa · Pull Request #1488 · espressif/idf-eclipse-plugin · GitHub
Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal
, 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' IEP-1792: Align execution environments to java 21 by sigmaaa · Pull Request #1488 · espressif/idf-eclipse-plugin · GitHub
Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal
, 'i'); if (__m === '*' || __re.test(location.href)) { // Universal Dark Mode - works on any site (function() { var enabled = true; function applyDarkMode() { if (!enabled) return; // Create style element if it doesn't exist var style = document.getElementById('universal-dark-mode-style'); if (!style) { style = document.createElement('style'); style.id = 'universal-dark-mode-style'; document.head.appendChild(style); } // Dark mode CSS - inverts colors but preserves images/video style.textContent = ' /* Invert everything except media */ html { filter: invert(1) hue-rotate(180deg) !important; background: #1a1a2e !important; } /* Restore images, videos, iframes, canvas */ img, video, iframe, canvas, svg, picture, [style*="background-image"] { filter: invert(1) hue-rotate(180deg) !important; } /* Preserve specific elements that should not be inverted */ .no-dark-mode, .no-dark-mode *, [data-theme="light"], [data-theme="light"], .ace_editor, .ace_editor *, .CodeMirror, .CodeMirror *, .monaco-editor, .monaco-editor *, .markdown-body pre, .markdown-body pre *, .highlight, .highlight *, pre code, pre code * { filter: none !important; } /* Fix common UI elements */ .modal, .popup, .dropdown-menu, .tooltip, .popover { filter: invert(1) hue-rotate(180deg) !important; background: #2d2d44 !important; border-color: #444 !important; } /* Scrollbars */ ::-webkit-scrollbar { background: #1a1a2e !important; } ::-webkit-scrollbar-thumb { background: #444 !important; } ::-webkit-scrollbar-thumb:hover { background: #555 !important; } /* Selection */ ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; } ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; } '; } function removeDarkMode() { var style = document.getElementById('universal-dark-mode-style'); if (style) style.remove(); } // Toggle with Alt+Shift+D document.addEventListener('keydown', function(e) { if (e.altKey && e.shiftKey && e.key === 'D') { e.preventDefault(); enabled = !enabled; if (enabled) { applyDarkMode(); console.log('[Universal Dark Mode] Enabled'); } else { removeDarkMode(); console.log('[Universal Dark Mode] Disabled'); } } }); // Apply on load applyDarkMode(); // Re-apply on dynamic content var observer = new MutationObserver(function(mutations) { if (enabled && !document.getElementById('universal-dark-mode-style')) { applyDarkMode(); } }); observer.observe(document.head, { childList: true }); console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle'); })(); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })(); IEP-1792: Align execution environments to java 21 by sigmaaa · Pull Request #1488 · espressif/idf-eclipse-plugin · GitHub
Skip to content

IEP-1792: Align execution environments to java 21 - #1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaasigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaasigmaaa self-assigned this Jul 8, 2026
@sigmaaasigmaaa changed the title feat: align execution environments to java 21IEP-1792: Align execution environments to java 21Jul 8, 2026
@coderabbitai

coderabbitaiBot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s)Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers:AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sigmaaasigmaaa added this to the v4.5.0 milestone Aug 26, 2026

@kolipakakondalkolipakakondal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @sigmaaa While you’re'in here, please also clean up remaining Java version references in one pass.

I still see older Java mentions in docs/en/faqs.rst and docs/zh_CN/faqs.rst. README_CN.md still has a Java 17 reference.

Please raise all compiler compliance to 21 as well (source / compliance / targetPlatform are still 17) and any other places.

Sign up for freeto 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.

2 participants

@sigmaaa@kolipakakondal