Skip to content

Conversation

@darvld
Copy link
Member

@darvld darvld commented Aug 18, 2025

Ready for review Powered by Pull Request Badge

Summary

Downgrades JLine to 3.28.0 to fix a silent crash when using KInquirer in the CLI (e.g. during elide init and other interactive commands).

@darvld darvld added the bug Something isn't working label Aug 18, 2025
@darvld darvld requested a review from sgammon as a code owner August 18, 2025 13:05
@darvld darvld force-pushed the fix/cli-silent-crash branch from b7210cd to fd86cc8 Compare August 19, 2025 21:45
@codecov
Copy link

codecov bot commented Aug 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.68%. Comparing base (fe71087) to head (6d3a928).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1612      +/-   ##
==========================================
- Coverage   39.68%   39.68%   -0.01%     
==========================================
  Files         781      781              
  Lines       37169    37169              
  Branches     5253     5253              
==========================================
- Hits        14752    14751       -1     
  Misses      20648    20648              
- Partials     1769     1770       +1     
Flag Coverage Δ
jvm 39.68% <ø> (-0.01%) ⬇️
lib 39.68% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file 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 3399f8f...6d3a928. 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.

Signed-off-by: Dario Valdespino <dvaldespino00@gmail.com>
@darvld darvld force-pushed the fix/cli-silent-crash branch from fd86cc8 to 6d3a928 Compare August 19, 2025 22:04
@darvld darvld merged commit b72d183 into main Aug 19, 2025
18 checks passed
@sgammon sgammon mentioned this pull request Sep 21, 2025
31 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants