Louis Maddox

The dep should be a dev dep if it's just for the example. I merged a PR for it in figment2 (fork): - https://github.com/lmmx/figment2/pull/2

Top level must be an object not a list

This behaviour you're seeing is what's documented in the README, here: https://github.com/sharkdp/bat/blob/006d77fa393172b7089da27b67d3e0b235363720/README.md?plain=1#L657-L667 > These options will not be added if: > - The `BAT_PAGER` environment variable contains any command-line arguments...

Hmm I thought I understood the issue and had repro'd, now I'm not sure. These 2 commands behave differently for me ```sh louis 🚶 ~/dev/bat $ echo foo | BAT_PAGER='less...

Got you, bat exitting shouldn't stop `less` basically. You should still be able to scroll around... I can do that when I run ```sh for i in {0..10}; do echo...

Possibly SIGINT is going to both bat and less, or the pipe handling is wrong, or something to do with the process that's called... Is it that `Command::spawn()` creates the...

I've tried debugging what's happening here. I confirmed that less receives SIGINT and exits with code 1 immediately. Without `-K`, less should interrupt its current operation but stay open for...

The report is about avoiding `-K` behaviour, adding `-K` would cause the immediate exit the user is trying to prevent: > bat makes less exit on ctrlc and I don't...

I think we want to ignore SIGINT while the pager is active, preventing our teardown from killing the pager, then restore the handler after less exits. `signal-hook` crate is the...

- Patched in #3447 Click to show minimal patch: ```diff diff --git a/Cargo.toml b/Cargo.toml index d64f3fcc..a5ecca9a 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -75,6 +75,7 @@ execute = { version =...