Skip to content

Conversation

@heckj
Copy link
Owner

@heckj heckj commented Feb 21, 2025

Description

tweak logging output to include host

Motivation and Context

makes it easier to track commands against multiple hosts

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected. Examples include renaming parameters in public API, removing public API, or adding a new parameter to public API without a default value.)

Checklist

  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have run ./scripts/preflight.bash and it passed without errors.

@heckj heckj self-assigned this Feb 21, 2025
@codecov
Copy link

codecov bot commented Feb 21, 2025

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 74.50%. Comparing base (2442239) to head (60be9fc).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
Sources/Formic/RemoteHost.swift 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage   74.50%   74.50%           
=======================================
  Files          25       25           
  Lines        1212     1212           
=======================================
  Hits          903      903           
  Misses        309      309           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@heckj heckj merged commit 628e8f1 into main Feb 21, 2025
3 checks passed
@heckj heckj deleted the logupdate branch February 21, 2025 23:13
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