Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: rewrite prefix after carriage return (#6989)
### Description Fixes #6969 If we encounter a `\r` in a task's outputs we make sure we rewrite our prefix so it doesn't get overwritten by the output that follows the `\r`. This does result in slightly more writing to stdout than strictly necessary on systems that use `CRLF` instead of just `LF` as we will now print out our prefix before processing `LF`. The user still sees the correct output and this keeps the implementation simple. [`\r` reference](https://en.wikipedia.org/wiki/Carriage_return#Computers) ### Testing Instructions Unit tests for a few scenarios, but primary way to test is to run `next` using `turbo`: ``` [0 olszewski@chriss-mbp] /tmp $ npx create-turbo@latest cr-test pnpm [0 olszewski@chriss-mbp] /tmp $ cd cr-test [0 olszewski@chriss-mbp] /tmp/cr-test $ pnpm rm turbo # make sure we're using locally built turbo ``` <img width="612" alt="Screenshot 2024-01-11 at 10 11 46 AM" src="https://github.com/vercel/turbo/assets/4131117/20ad5bfd-176c-486b-8f05-d77206168af0"> Closes TURBO-2030
- Loading branch information
82c2622
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Successfully deployed to the following URLs:
turbo-site – ./docs
turbo.vercel.sh
turborepo.com
turborepo.org
turbo.vercel.app
turbo.build
www.turbo.build
turbopack.org
www.turbopack.org
www.turborepo.com
www.turborepo.org
turbo-site.vercel.sh
turbo-site-git-main.vercel.sh