Skip to content

Conversation

@CriszelGipala
Copy link
Contributor

Changes:

  • Fix 11 broken documentation links: typos (docs.source to docs/source), incorrect filenames (system-connection-* to system-* ), and outdated references to renamed/removed commands (generate kube, play kube, manifest subcommands).

Why:

  • Multiple links in the Docker-to-Podman command mapping table lead to non-existent pages, making it harder for new users to find the correct documentation.

Scope: transfer.md only - no code changes.

Checklist

Ensure you have completed the following checklist for your pull request to be reviewed:

  • Certify you wrote the patch or otherwise have the right to pass it on as an open-source patch by signing all
    commits. (git commit -s). (If needed, use git commit -s --amend). The author email must match
    the sign-off email address. See CONTRIBUTING.md
    for more information.
  • Referenced issues using Fixes: #00000 in commit message (if applicable)
  • Tests have been added/updated (or no tests are needed)
  • Documentation has been updated (or no documentation changes are needed)
  • All commits pass make validatepr (format/lint checks)
  • Release note entered in the section below (or None if no user-facing changes)

Does this PR introduce a user-facing change?


@CriszelGipala CriszelGipala force-pushed the cgipala-fix-typos-and-links branch from b4d2e3c to 2045e23 Compare February 9, 2026 12:38
Copy link
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

Thanks for the changes. I found several broken links, and it seems some of the fixes broke lists that were previously working.

@CriszelGipala CriszelGipala force-pushed the cgipala-fix-typos-and-links branch 2 times, most recently from 6dea26f to d10fa51 Compare February 10, 2026 01:28
Copy link
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

Thanks, last two nits.

Copy link
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

LGTM, just one last thing: squash your commits. Thanks

@packit-as-a-service
Copy link

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

Changes:
- Fix 'docs.source' typo to 'docs/source' on lines 54, 59, 60, 65.
- Correct filenames on lines 84, 91, 170, 193-196 to point to files that actually exist in the repo.
- Removed line 139 as the command no longer exists.

Why:
- The linked documentation filenames were renamed resulting in broken links.

Scope:
- transfer.md only - no code changes

Signed-off-by: Criszel Gipala <[email protected]>
@CriszelGipala CriszelGipala force-pushed the cgipala-fix-typos-and-links branch from 320e5d9 to 1243b2b Compare February 10, 2026 13:47
Copy link
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

LGTM

@Honny1 Honny1 merged commit 2b9e652 into containers:main Feb 11, 2026
24 of 40 checks passed
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