Skip to content

feat: add -F flag for custom SSH config file#68

Merged
psviderski merged 2 commits into
psviderski:mainfrom
tekjar:main
Apr 21, 2026
Merged

feat: add -F flag for custom SSH config file#68
psviderski merged 2 commits into
psviderski:mainfrom
tekjar:main

Conversation

@tekjar

@tekjar tekjar commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Add -F option to specify a custom SSH config file path, matching ssh's standard -F flag behavior.

  • Parse and validate -F option argument
  • Pass -F to ssh commands when specified
  • Update README with custom SSH config file examples
  • Add -F to help text and usage examples

Add -F option to specify a custom SSH config file path, matching ssh's
standard -F flag behavior.

- Parse and validate -F option argument
- Pass -F to ssh commands when specified
- Update README with custom SSH config file examples
- Add -F to help text and usage examples

@psviderski psviderski left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good, thank you for putting this together!
Just one minor comment to add a long-form --ssh-config flag and I'm happy to merge this after that.

Comment thread docker-pussh Outdated
echo ""
echo "Options:"
echo " -h, --help Show this help message."
echo " -F path Path to SSH config file."

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Let's also add a long-form option --ssh-config for consistency with other flags

- Add `--ssh-config` as primary long-form option for SSH config path
- Keep `-F` as short alias for backward compatibility
- Update help text and error messages to reflect new option name
- Update usage example to use `--ssh-config` instead of `-F`
@psviderski
psviderski merged commit 5d3e348 into psviderski:main Apr 21, 2026
1 check passed
@tekjar

tekjar commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

Thank you :)

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