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

nix repl: hide progress bar during :edit #10618

Merged
merged 1 commit into from
Apr 29, 2024
Merged

Conversation

srhb
Copy link
Contributor

@srhb srhb commented Apr 28, 2024

Motivation

Attempt to fix #10617

Context

I have no idea what I'm doing, so someone familiar with, like, C++, or how the progress bar usually works, should probably see if this is sensible. It just looked vaguely-like-the-right-spot to do the thing. 😸

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@srhb srhb requested a review from edolstra as a code owner April 28, 2024 14:35
@github-actions github-actions bot added the repl The Read Eval Print Loop, "nix repl" command and debugger label Apr 28, 2024
src/libcmd/repl.cc Show resolved Hide resolved
src/libcmd/repl.cc Outdated Show resolved Hide resolved
@srhb
Copy link
Contributor Author

srhb commented Apr 29, 2024

@edolstra Thanks, squashed in your suggestions! 👍

@srhb
Copy link
Contributor Author

srhb commented Apr 29, 2024

Re-tested, still seems good!

@edolstra edolstra merged commit 4161f3c into NixOS:master Apr 29, 2024
9 checks passed
@srhb srhb deleted the editor-pause-repl branch April 29, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
repl The Read Eval Print Loop, "nix repl" command and debugger
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nix repl :editor messes up the terminal
2 participants