Canop / bacon

background rust code check
https://dystroy.org/bacon
GNU Affero General Public License v3.0
1.96k stars 73 forks source link

fix either raw/standard output not rewrapped #235

Closed Canop closed 1 month ago

Canop commented 1 month ago

Fix #234

teor2345 commented 1 month ago

Thanks for the quick fix, but this doesn't seem to quite work the way I expect.

When I set wrap = false in prefs.toml, no wrapping happens at all, either in the initial test screen, or the fails result summary screen, regardless of how many times I press w. I expected that w would cause wrapping (and then unwrapping when pressed again), in both screens.

When I set wrap = true in prefs.toml, wrapping happens in the fails screen, and correctly unwraps on w (and re-wraps on another w). This works as expected. But the raw output starts unwrapped, and doesn't wrap on w.

Is there another way of switching back to the detailed output that also does wrapping? Because maybe I'm expecting the wrong things from raw mode. If that's the case, then the only bug is disabling wrapping entirely in the fails screen when wrap = false.

Canop commented 1 month ago

maybe I'm expecting the wrong things from raw mode

No, I'm just retarded, and tired, and I broke the fix before pushing it...

Canop commented 1 month ago

I think I fixed the fix.

teor2345 commented 1 month ago

Sory for the delay, yes this works as expected!