Warning: This build has drifted.
The coverage report for this pull request build may be inaccurate because its base commit is no longer the HEAD of its target branch.
This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

    • Learn more: For more information on this, see Tracking coverage changes for pull request builds.
    • Fix now: For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
    • Prevent going forward: To avoid this issue with future PRs, see these Recommended CI Configurations.

fix: address go-reviewer findings from stability fixes

- Simplify Close() to delegate to Stop() instead of duplicating the
  channel-close guard (pinger.go)
- Eliminate data race in Start(): store results slice inside PortChecker
  so Start() never reads t.PortResults directly (portchecker.go)
- Handle header write error in setupLogger; return error instead of
  silently dropping it (mping.go)
- Unify ipv4/ipv6 conflict error message between mergeHosts and parseArgs
  (mping.go)
- Add regression tests: setupLogger with existing file must not duplicate
  header; mergeHosts must reject ipv4+ipv6 in YAML (mping_test.go)