Skip to content

Conversation

@sgammon
Copy link
Member

@sgammon sgammon commented Nov 10, 2025

Ready for review Powered by Pull Request Badge

Summary

Fixes a suite of Kotlin Compiler bugs/failures currently extant on main; rolls back to 2.2.21. Disables TruffleRuby until lang initialization is cleaned up.

@sgammon sgammon added this to the Release R18: Beta milestone Nov 10, 2025
@sgammon sgammon requested a review from a team November 10, 2025 18:36
@sgammon sgammon self-assigned this Nov 10, 2025
@sgammon sgammon requested a review from dzodkin33 November 10, 2025 18:36
@sgammon sgammon added bug Something isn't working lang:kotlin Related to Kotlin lang support labels Nov 10, 2025
@sgammon sgammon added this to Elide Nov 10, 2025
@sgammon sgammon moved this to In Progress in Elide Nov 10, 2025
@sgammon sgammon moved this from In Progress to Done in Elide Nov 10, 2025
Signed-off-by: Sam Gammon <sam@elide.dev>
Signed-off-by: Sam Gammon <sam@elide.dev>
Signed-off-by: Sam Gammon <sam@elide.dev>
@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.92%. Comparing base (7db864e) to head (9a2ad5e).
⚠️ Report is 26 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1739      +/-   ##
==========================================
+ Coverage   40.23%   42.92%   +2.69%     
==========================================
  Files         888      895       +7     
  Lines       37088    42363    +5275     
  Branches     5684     5940     +256     
==========================================
+ Hits        14922    18186    +3264     
- Misses      20229    21982    +1753     
- Partials     1937     2195     +258     
Flag Coverage Δ
jvm 42.92% <ø> (+2.69%) ⬆️
lib 42.92% <ø> (+2.69%) ⬆️

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

Files with missing lines Coverage Δ
...kotlin/elide/tool/cli/cmd/repl/ToolShellCommand.kt 30.60% <ø> (+0.20%) ⬆️
.../kotlin/elide/runtime/gvm/kotlin/KotlinLanguage.kt 0.00% <ø> (ø)
.../src/main/kotlin/elide/tooling/jvm/JvmLibraries.kt 0.00% <ø> (ø)

... and 572 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d2f303...9a2ad5e. Read the comment docs.

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

@sgammon sgammon merged commit 18b5418 into main Nov 10, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lang:kotlin Related to Kotlin lang support

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Kotlin compiler failing on main Kotlinc: Elide kotlinc fails when compiling kotlin sources that reference java sources

3 participants