-
Notifications
You must be signed in to change notification settings - Fork 584
Open
Open
Copy link
Labels
nextItems in scope for the next few milestonesItems in scope for the next few milestonesuxUser experience features and fixes.User experience features and fixes.
Description
Feature or enhancement request details
Right now some output that should probably go to stderr is just being printed with print().
All commands should log diagnostics to stderr and print data proper to stdout.
For this we could create a plaintext-to-stderr LogHandler in ContainerClient that all commands use.
At present it appears we do no logging in ContainerClient, with the single Exception of Sources/ContainerClient/ExitMonitor.swift, which is actually only used on the server side. Therefore, we can limit the scope of this to just going through each of the command groups.
Code of Conduct
- I agree to follow this project's Code of Conduct
Metadata
Metadata
Assignees
Labels
nextItems in scope for the next few milestonesItems in scope for the next few milestonesuxUser experience features and fixes.User experience features and fixes.