Skip to content

Add updated Graal version#7777

Merged
madurangasiriwardena merged 1 commit intowso2:masterfrom
madurangasiriwardena:graal-upgrade
Mar 3, 2026
Merged

Add updated Graal version#7777
madurangasiriwardena merged 1 commit intowso2:masterfrom
madurangasiriwardena:graal-upgrade

Conversation

@madurangasiriwardena
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 26, 2026 04:20
@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7700b49 and 290ef36.

📒 Files selected for processing (1)
  • pom.xml

📝 Walkthrough

Walkthrough

The pull request updates GraalVM version properties from 22.3.4.wso2v1 to 25.0.2.wso2v1 with corresponding version range adjustments in pom.xml. Additionally, ICU-related configuration settings and associated comments are removed from the file.

Changes

Cohort / File(s) Summary
Dependency Version Updates
pom.xml
Updated GraalVM version from 22.3.4.wso2v1 to 25.0.2.wso2v1 and adjusted version range from [22.3.0,23.0.0) to [25.0.0,26.0.0). Removed ICU version property (71.1) and associated comment block.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is entirely empty with no information provided, while the repository template requires comprehensive documentation across multiple sections. Add a complete PR description following the template, at minimum covering Purpose, Goals, Approach, and Test environment sections to explain the upgrade rationale and testing performed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change in the PR, which updates GraalVM version from 22.3.4.wso2v1 to 25.0.2.wso2v1 in pom.xml.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the parent Maven configuration to use a newer WSO2-packaged GraalVM version, which affects how Graal-related artifacts (e.g., graal-sdk) are resolved across modules.

Changes:

  • Bumped graalvm.wso2.version from 22.3.4.wso2v1 to 25.0.2.wso2v1.
  • Updated the GraalVM version range to [25.0.0,26.0.0).
  • Removed the icu.version property (and its associated comment), which appears unused in the repo.

<!-- ICU version must be reflecting graalvm version -->
<icu.version>71.1</icu.version>
<graalvm.wso2.version>25.0.2.wso2v1</graalvm.wso2.version>
<graalvm.version.range>[25.0.0,26.0.0)</graalvm.version.range>
Copy link

Copilot AI Feb 26, 2026

Choose a reason for hiding this comment

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

graalvm.version.range is defined/updated here but appears unused across the repository (no references to ${graalvm.version.range} or the property name). Consider either wiring it into the relevant OSGi import/export version constraints (if that was the intent) or removing the property to avoid confusion and future drift.

Suggested change
<graalvm.version.range>[25.0.0,26.0.0)</graalvm.version.range>

Copilot uses AI. Check for mistakes.
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.41%. Comparing base (7700b49) to head (290ef36).
⚠️ Report is 60 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #7777      +/-   ##
============================================
+ Coverage     52.31%   52.41%   +0.09%     
+ Complexity    19862    19826      -36     
============================================
  Files          2130     2130              
  Lines        124116   123429     -687     
  Branches      24549    24460      -89     
============================================
- Hits          64935    64693     -242     
+ Misses        51181    50768     -413     
+ Partials       8000     7968      -32     
Flag Coverage Δ
unit 36.87% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@madurangasiriwardena madurangasiriwardena merged commit 6613bcc into wso2:master Mar 3, 2026
10 of 11 checks passed
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