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. Co-authorship added to the shared change here in this PR.]

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
Contributor

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. :)

@natevw
Copy link
Author

natevw commented Feb 19, 2026

iiuc adding co-authorship would entail:

  • gathering Simone's byline (preferred name/email) from one of their other published commits
  • amending the shared commit's message to add a line with the Co-Authored-By: Name <name@email.com> convention
  • force pushing those changes to this PR branch

(update: this does seem aligned with GH's documentation on the subject)

Happy to do so, if a force push wouldn't inconvenience the maintainers here too badly?

@Ronitsabhaya75
Copy link
Contributor

@natevw I'm not really sure for force push tbh prolly john would know it

@saehejkang
Copy link
Contributor

Happy to do so, if a force push wouldn't inconvenience the maintainers here too badly?

Feel free to force push whenever, as it is your feature branch. It is encouraged when you are cleaning up your commit history (rebasing, squashing, fixing commit messages). In this case, you are amending a commit to add someone as a co-author so it warrants a force push.

natevw and others added 2 commits February 19, 2026 19:40
@natevw
Copy link
Author

natevw commented Feb 20, 2026

Co-authorship added, looks like it must have worked here in the Github UI above :-)

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