Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Prompt Logger #4134

Merged
merged 1 commit into from
Dec 14, 2024
Merged

Fix Prompt Logger #4134

merged 1 commit into from
Dec 14, 2024

Conversation

lihaoyi
Copy link
Member

@lihaoyi lihaoyi commented Dec 14, 2024

Fixes #4133

The problem was that the cleanup-process-folder logic running in ServerLauncher#run was taking place after the terminfo file was initially created in runTermInfoThread, resulting in it wiping out the terminfo file that then never gets re-created until you resize the terminal.

This PR moves the cleanup logic to always run just before the terminfo file is created, renaming runTermInfoThread into prepareMillRunFolder and making it responsible for both cleanup of the mill process folder as well as initialization with new terminfo metadata

@lihaoyi lihaoyi marked this pull request as ready for review December 14, 2024 12:04
@lihaoyi lihaoyi merged commit e4af057 into com-lihaoyi:main Dec 14, 2024
26 checks passed
@lefou lefou added this to the 0.12.4 milestone Dec 14, 2024
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.

Prompt logger seems to be broken in main
2 participants