-
Notifications
You must be signed in to change notification settings - Fork 388
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
Improved CLI: stdout streaming support #7094
Conversation
Disagree with this aspect. Being able to output to stdout is great, but I don't think it should be required. Being explicit with -o is frequently more clear, esp in contexts where you aren't really sure about how your shell is going to handle pipes (thinking pixi, github runners, etc.) |
cb7fbda
to
98dcaee
Compare
I agree with your disagreement 😄 I only removed that feature because it lead to very weird interaction with another feature that I didn't publish in the end. I'll add those back. |
e7844f9
to
8d4e1ef
Compare
f73f98e
to
55e7b1a
Compare
You can now do this:
Also made sure that encoders and decoders accurately report the size of the data they plow through.
Checklist
main
build: rerun.io/viewernightly
build: rerun.io/viewerCHANGELOG.md
and the migration guideTo run all checks from
main
, comment on the PR with@rerun-bot full-check
.