Skip to content

Update macOS installation instructions for Podman#494

Merged
TomSweeneyRedHat merged 2 commits into
containers:mainfrom
docdeveloper:patch-1
Jun 24, 2026
Merged

Update macOS installation instructions for Podman#494
TomSweeneyRedHat merged 2 commits into
containers:mainfrom
docdeveloper:patch-1

Conversation

@docdeveloper

Copy link
Copy Markdown
Contributor

Apply good tech writing practices:

  • Use a procedure with steps
  • Use active voice
  • Remove duplicated content
  • Remove redundancy
  • Improve clarity

Apply good tech writing practices:
- Use a procedure with steps
- Use active voice
- Remove duplicated content
- Remove redundancy
- Improve clarity

Signed-off-by: Michal Skowron <mskowron@guidewire.com>
Comment thread docs/installation.md Outdated
Comment thread docs/installation.md
Podman can be downloaded from the [Podman.io](https://podman.io) website.
You can get binaries and a pkginstaller from our [GitHub release page](https://github.com/containers/podman/releases).
</details>
- Not recommended

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

not necessary, but a line feed above this makes it clearer to review IMO

Comment thread docs/installation.md Outdated
Comment thread docs/installation.md Outdated
@TomSweeneyRedHat

Copy link
Copy Markdown
Member

@docdeveloper thanks for the updates! A few comments to consider.

Signed-off-by: Michal Skowron <mskowron@guidewire.com>
@TomSweeneyRedHat

Copy link
Copy Markdown
Member

Thanks @docdeveloper !

@TomSweeneyRedHat

Copy link
Copy Markdown
Member

LGTM

@TomSweeneyRedHat

Copy link
Copy Markdown
Member

/lgtm

@TomSweeneyRedHat TomSweeneyRedHat merged commit 25e916d into containers:main Jun 24, 2026
3 checks passed
@docdeveloper

Copy link
Copy Markdown
Contributor Author

@TomSweeneyRedHat , thank you merging the PR.
I just had a look at the published docs and some parts don't look right:

  1. The procedure for installaton on macOS isn't a numbered list.
  2. Formatting in expandable sections is incorrect. The output didn't render properly from Markdown.

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.

2 participants