Skip to content

Update piping alias section for clarity#2124

Open
c-git wants to merge 3 commits intonushell:mainfrom
c-git:patch-1
Open

Update piping alias section for clarity#2124
c-git wants to merge 3 commits intonushell:mainfrom
c-git:patch-1

Conversation

@c-git
Copy link
Contributor

@c-git c-git commented Feb 25, 2026

Clarify that aliases do not support multiple commands and provide examples for defining commands without parameters.

Clarify that aliases do not support multiple commands and provide examples for defining commands without parameters.
@c-git
Copy link
Contributor Author

c-git commented Feb 25, 2026

Will try to clean it up a bit after work today. Too tired to ensure it's clear right now.

Add the intention that is we are attempting to achieve in the example.
@c-git
Copy link
Contributor Author

c-git commented Feb 25, 2026

I went back over it but needed to make more changes so might be best to come back to it again.

Clarified explanation of alias command behavior and output.
@c-git
Copy link
Contributor Author

c-git commented Feb 25, 2026

Still ended up making another change, so will try again another time to read it over.

Once I'm finally done, do I need to squash the commits or will you squash on merge?

@c-git
Copy link
Contributor Author

c-git commented Mar 2, 2026

Considered capitalizing "not" in the first change but decided against it as I think it's clear enough and thus made no changes and I think it's ok as is. Let me know if anything could be improved.

Extract of the line I considered changing (with the change made) in case it would improve clarity or readability.

Note aliases currently do NOT support including multiple commands. So for example alias uuidgen = uuidgen | tr A-F a-f (to make uuidgen on mac behave like linux) won't work.
The solution is to define a command without parameters that calls the system program uuidgen via ^.

@c-git c-git marked this pull request as ready for review March 2, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant