Skip to content

Remove '-o' shorthand for stdout from script_add_args#11043

Merged
Wumpf merged 2 commits intorerun-io:mainfrom
karimknaebel:patch-1
Aug 29, 2025
Merged

Remove '-o' shorthand for stdout from script_add_args#11043
Wumpf merged 2 commits intorerun-io:mainfrom
karimknaebel:patch-1

Conversation

@karimknaebel
Copy link
Contributor

Related

What

The -o shorthand tends to clash with people's CLI args. Single-letter flags feel a bit overkill here.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Thanks for opening this pull request.

Because this is your first time contributing to this repository, make sure you've read our Contributor Guide and Code of Conduct.

Copy link
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, I think you're right!
Could you please add a brief note about this in docs/content/reference/migration/migration-0-25.md? Thank you!

@karimknaebel
Copy link
Contributor Author

done

Copy link
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx

@Wumpf Wumpf merged commit d8bed36 into rerun-io:main Aug 29, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants