go-octopusdeploy
go-octopusdeploy copied to clipboard
support for automation environment and tool data on useragent
[sc-31593]
This pull request has been linked to Shortcut Story #31593: Add telemetry to the CLI vNext.
Thinking out loud here: Maybe we should consider using context
instead of parameter passing for the tool information? A good rule of thumb is that any data required for a function to run should be passed as parameters. In this case (however), it's contextual info.
@domenicsim1 thoughts?
Maybe we should consider using
context
instead of parameter passing for the tool information? A good rule of thumb is that any data required for a function to run should be passed as parameters. In this case (however), it's contextual info.
I am by far not an expert here, but it seems that context is analogous to a cancellation token, with the ability to pass key/value data with it, for example the context of a http request that allows the request to be cancelled during execution.