Skip to content

Conversation

@sgammon
Copy link
Member

@sgammon sgammon commented Jul 16, 2025

Ready for review Powered by Pull Request Badge

Summary

Adds interop smoke tests for Python with generator functions; fixes a condition where the Python context could not be created because lang configuration options had not initialized yet.

Changelog

fix(python): create context via `PythonContext.get(null`)
chore: add python generator sample

fix(python): create context via `PythonContext.get(null`)
chore: add python generator sample

Signed-off-by: Sam Gammon <sam@elide.ventures>
@sgammon sgammon added this to the Release R18: Beta milestone Jul 16, 2025
@sgammon sgammon self-assigned this Jul 16, 2025
@sgammon sgammon added bug Something isn't working lang:python Issues relating to Python labels Jul 16, 2025
@sgammon sgammon added this to Elide Jul 16, 2025
@sgammon sgammon moved this to In Progress in Elide Jul 16, 2025
@codecov
Copy link

codecov bot commented Jul 16, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 40.67%. Comparing base (385ef1d) to head (bb4cb7e).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...untime/lang/javascript/ElideInteropModuleLoader.kt 50.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1543      +/-   ##
==========================================
+ Coverage   40.43%   40.67%   +0.23%     
==========================================
  Files         749      749              
  Lines       35329    35332       +3     
  Branches     4968     4968              
==========================================
+ Hits        14287    14372      +85     
+ Misses      19340    19246      -94     
- Partials     1702     1714      +12     
Flag Coverage Δ
jvm 40.67% <50.00%> (+0.23%) ⬆️
lib 40.67% <50.00%> (+0.23%) ⬆️

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

Files with missing lines Coverage Δ
...untime/lang/javascript/ElideInteropModuleLoader.kt 79.06% <50.00%> (+65.57%) ⬆️

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 385ef1d...bb4cb7e. 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 ea278e3 into main Jul 16, 2025
27 of 28 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Elide Jul 16, 2025
@elidebot elidebot mentioned this pull request Jul 19, 2025
29 tasks
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:python Issues relating to Python

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants