Skip to content

Conversation

@ncode
Copy link
Owner

@ncode ncode commented Jan 29, 2026

This pull request introduces several improvements and new features, focusing on enhanced host specification parsing, improved documentation, and better test coverage. The most significant changes are the addition of robust host parsing logic, comprehensive tests for host parsing and configuration, and a major update to the project documentation. Additionally, new GitHub Actions workflows for CI, Go build/test, and CodeQL analysis have been added.

Host specification and parsing improvements:

  • Added cmd/hosts.go with robust parsing for host specifications, including support for user@host, host:port, IPv6, and group/user precedence, along with error handling and utility functions.
  • Refactored cmd/root.go to use the new host parsing logic, ensuring consistent handling of positional hosts, host groups, and hosts files.

Testing enhancements:

  • Added cmd/hosts_test.go with comprehensive unit tests for host parsing, group parsing, file parsing, argument parsing, and prompt flag override logic.

Documentation improvements:

  • Rewrote and expanded README.md to provide clear installation instructions, configuration schema, usage examples, host specification formats, flag descriptions, interactive commands, and testbed instructions.

Project structure and CI:

  • Moved internal packages to internal/ for better code organization and updated imports accordingly.
  • Added GitHub Actions workflows for CI/testing (.github/workflows/ci.yml), Go build/test (.github/workflows/go.yml), and CodeQL analysis (.github/workflows/codeql.yml). [1] [2] [3]

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@ncode ncode merged commit 96b52de into master Jan 29, 2026
2 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