Skip to content

Conversation

Managor
Copy link
Member

@Managor Managor commented Sep 5, 2025

I'm looking for feedback on where this might not apply and what else to add to the list.

@github-actions github-actions bot added the documentation Issues/PRs modifying the documentation. label Sep 5, 2025
@Managor Managor marked this pull request as draft September 5, 2025 16:38
@Managor Managor mentioned this pull request Sep 5, 2025
6 tasks
Copy link
Member

@dmmqz dmmqz left a comment

Choose a reason for hiding this comment

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

I would add an example, it's pretty unclear what this is about without context.

@Managor
Copy link
Member Author

Managor commented Sep 5, 2025

One thing I'm having trouble placing is data redirection from a file if the contributor desires to use it. < file.txt can be placed anywhere in the command.

The most logical place would be to put it right after the command so that it points towards the command it's being directed to, like this command < file.txt subcommand --option ..., but I'm having doubts

Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the addition. I think the order for data redirection to a file from a file being last is fine, since we follow the same in a majority of our pages for outputting to a file.

@Managor
Copy link
Member Author

Managor commented Sep 5, 2025

@kbdharun You misunderstand. I'm talking about the data redirection from a file not to a file.

@Managor
Copy link
Member Author

Managor commented Sep 9, 2025

I guess it's ready.

@Managor Managor marked this pull request as ready for review September 9, 2025 11:13
Copy link
Member

@MachiavelliII MachiavelliII left a comment

Choose a reason for hiding this comment

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

Brilliant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues/PRs modifying the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants