Skip to content

Conversation

@devop-mmcgrath
Copy link

@devop-mmcgrath devop-mmcgrath commented Jun 12, 2025

Does this PR introduce a user-facing change?

None

@openshift-ci openshift-ci bot added the do-not-merge/release-note-label-needed Enforce release-note requirement, even if just None label Jun 12, 2025
@packit-as-a-service
Copy link

[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore.

@openshift-ci openshift-ci bot added release-note-none and removed do-not-merge/release-note-label-needed Enforce release-note requirement, even if just None labels Jun 12, 2025
@baude
Copy link
Member

baude commented Jun 12, 2025

looks like you got it @devop-mmcgrath !

LGTM

/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 12, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude, devop-mmcgrath

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 12, 2025
@kolyshkin
Copy link
Contributor

@devop-mmcgrath can you please squash the two commits?

@mheon
Copy link
Member

mheon commented Jun 13, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 13, 2025
The --pod flag is important for users working with pods but lacked
documentation examples. Added examples showing:
- Basic --pod usage to display pod information
- Using --pod with -a to show all containers and their pods
- Filtering containers by pod name
- Custom formatting with pod-related placeholders

Also: removed trailing whitespace on a few lines

Fixes containers#26367
Assisted-by: Claude Sonnet 4

Removed trailing whitespace on a few lines

Signed-off-by: Mike McGrath <[email protected]>

MH: Squashed, force-pushed to reset CI
Signed-off-by: Matt Heon <[email protected]>
@mheon mheon force-pushed the fix-stats-manpage-format branch from f732cea to ffa4c12 Compare June 13, 2025 09:15
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jun 13, 2025
@mheon
Copy link
Member

mheon commented Jun 13, 2025

Retriggered CI
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 13, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 2708735 into containers:main Jun 13, 2025
31 of 36 checks passed
@stale-locking-app stale-locking-app bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 12, 2025
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Sep 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. release-note-none

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants