Skip to content

Add ps alias to ContainerList command#1226

Open
natevw wants to merge 2 commits intoapple:mainfrom
natevw:patch-1
Open

Add ps alias to ContainerList command#1226
natevw wants to merge 2 commits intoapple:mainfrom
natevw:patch-1

Conversation

@natevw
Copy link

@natevw natevw commented Feb 17, 2026

Presumed fix for #1213. [UPDATE: oops — did not see the existing #1217 until later. this is the same code change plus a corresponding doc edit.]

Full disclosure: I just did this via the GH web editor, did not set up a build environment or confirm the linter/formatter was happy with my version of the syntax. (My original intent in pulling it up was only to peek into what would be involved so that I could subconsciously mull over it while I psyched myself up to upgrade my machine to Liquid Glass. As it turns out, it didn't look like there was much to mull over!)

Updated the reference guide but I didn't see any related existing tests and as a simple alias seemed like something the command parser library would already have sufficient coverage on.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Motivation and Context

The docker equivalent to listing containers uses the ps command. This adds an alias for muscle memory compatibility.

Testing

  • Tested locally
  • Check for relevant tests
  • Added/updated docs

@natevw natevw marked this pull request as ready for review February 18, 2026 00:07
@saehejkang
Copy link
Contributor

[UPDATE: oops — did not see the existing https://github.com//pull/1217 until later. this is the same code change plus a corresponding doc edit.]

The doc edit is a good catch and it is worthwhile to get that updated with the new alias.

@simone-panico Are you cool with making the doc update in #1217 and then giving @natevw credit with a co-author commit? We can then focus on merging that PR and close this one.

@simone-panico
Copy link

Hey @natevw and @saehejkang,

I think we should merge this PR, it was your idea in the first place 😉 I will close my PR

@natevw
Copy link
Author

natevw commented Feb 18, 2026

Thanks @simone-panico and appreciate your work on that other issue I filed too! @saehejkang looks like this is the one to review now.

@Ronitsabhaya75
Copy link
Contributor

@natevw i think you give add @simone-panico credit as co author for work. :)

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.

4 participants

Comments