Skip to content

Conversation

@DevLev100
Copy link
Contributor

@DevLev100 DevLev100 commented Sep 17, 2025

Add openjdk25 and update openjdk to 25.

JDK25 was released on 16.09.2025.

Summary by CodeRabbit

  • New Features

    • Added a dedicated OpenJDK 25 package with automatic PATH and JAVA_HOME configuration.
  • Chores

    • Upgraded the main OpenJDK package to version 25.
    • Updated download links, checksums, and extraction directory for the new release.
    • Refreshed version-checking to track JDK 25.
    • Aligned autoupdate settings with the new version.

@coderabbitai
Copy link

coderabbitai bot commented Sep 17, 2025

Walkthrough

Updates the existing OpenJDK Scoop manifest to version 25 and adds a dedicated openjdk25.json manifest. Changes include updated download URLs, SHA256 checksums, extract directory, and checkver URL. Autoupdate and regex patterns remain consistent with prior configurations.

Changes

Cohort / File(s) Summary
OpenJDK manifest bump
bucket/openjdk.json
Bumps version from 24.0.2-12 to 25; updates 64-bit URL, SHA256, extract_dir to jdk-25; updates checkver.url to jdk.java.net/25; retains existing regex and autoupdate structure.
New OpenJDK 25 manifest
bucket/openjdk25.json
Adds standalone manifest for OpenJDK 25 with metadata, 64-bit URL and hash, extract_dir jdk-25, PATH and JAVA_HOME settings, checkver against jdk.java.net/25 with regex/replace, and autoupdate templates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A carrot-orange sun greets JDK twenty-five,
I thump my paws—new bins arrive!
Fresh URLs, crisp SHAs aligned,
PATHs hop true, JAVA_HOME defined.
In burrows deep, builds thrive—
Scoop, scoop, hooray—we keep code alive! 🥕🐇

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


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 and usage tips.

Pre-merge checks

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description gives a brief summary of the change and confirms the contributor read the contributing guide, but it does not follow the repository's required template: it is missing the required "Closes #XXXX" or "Relates to #XXXX" line and does not state that a relevant issue was opened or that similar issues/PRs were searched for, so the description is incomplete relative to the template. Add a "Closes #" or "Relates to #" line to the PR description and confirm (or provide a link to) the required Package Request or Bug Report that was opened before creating this PR; also state that you searched for similar issues/PRs and consider adding brief testing notes or compatibility information.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title "Add openjdk25" is concise and correctly describes the addition of a new openjdk25 manifest included in this changeset, and it is specific rather than generic; however it omits the related change that updates the existing openjdk manifest to version 25, so it only partially summarizes the PR's full scope. Overall the title refers to a real and significant part of the changeset and is acceptable as a short summary.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

openjdk

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

openjdk25

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

Check the full log for details.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (2)
bucket/openjdk.json (1)

4-4: Include GA build in version (e.g., 25-36) to align with checkver.replace.

checkver produces "${version}-${build}" (e.g., 25-36). Keeping "version": "25" causes a mismatch and noisier updates.

-    "version": "25",
+    "version": "25-36",
bucket/openjdk25.json (1)

1-6: Pin version to GA build (25-36) for accuracy and consistency.

-    "version": "25",
+    "version": "25-36",

Optional: align description wording with openjdk.json for consistency.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2c2afb3 and b2f1844.

📒 Files selected for processing (2)
  • bucket/openjdk.json (1 hunks)
  • bucket/openjdk25.json (1 hunks)
🔇 Additional comments (7)
bucket/openjdk.json (3)

12-12: LGTM: extract_dir matches ZIP root (jdk-25).


18-18: LGTM: checkver.url updated to 25 page; regex/replace remain compatible.


8-9: Verified — URL and SHA256 match.
Downloaded the artifact and computed sha256 = 85bcc178461e2cb3c549ab9ca9dfa73afd54c09a175d6510d0884071867137d3 (matches openjdk.json); archive top-level dir = "jdk-25".

bucket/openjdk25.json (4)

12-16: LGTM: extract_dir, PATH, and JAVA_HOME are correct.


17-21: LGTM: checkver regex/replace will yield 25-; matches autoupdate expectations.


22-32: LGTM: autoupdate templates and hash source are correct; extract_dir uses $matchVersion → jdk-25.


7-11: Verify URL and SHA‑256 for bucket/openjdk25.json

Compute the SHA‑256 of the artifact at the URL and confirm it equals 85bcc178461e2cb3c549ab9ca9dfa73afd54c09a175d6510d0884071867137d3 (same as openjdk.json); also list the zip root entries to confirm expected JDK layout.

Run:
python - <<'PY'
import hashlib,zipfile
fn='openjdk25.zip'
h=hashlib.sha256()
with open(fn,'rb') as f:
for b in iter(lambda: f.read(8192), b''):
h.update(b)
print(h.hexdigest())
with zipfile.ZipFile(fn) as z:
print(sorted({p.split('/')[0] for p in z.namelist() if '/' in p})[:20])
PY

@se35710 se35710 merged commit 355506a into ScoopInstaller:master Sep 18, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants