Chad Retz

I don't think we have plans to support `ddtrace` specifically at this time. I do think you need to pass through the import of `ddtrace` if it is used in...

Saw #2166 was marked as dupe. I do want to make sure there is a clear difference between "run any code before start/signal/update" and "initialize state before start/signal/update". I think...

Similarly the scope of the methods inside the `Struct`/`Data` block is as if you're part of the outer class/module. It treats the methods as nested methods instead of methods on...

> create two methods: The linked PR at https://github.com/temporalio/sdk-go/pull/1371 created `AssertActivityNumberOfCalls` and `AssertWorkflowNumberOfCalls`, can you confirm they work for your use case?

https://github.com/temporalio/sdk-go/pull/1371 was maybe developed wrong. @Quinn-With-Two-Ns - thoughts?

Definitely don't want to re-introduce the performance issue this solves. It's just unfortunate that the only way to fix is to set an environment variable that is commonly used by...

I am concerned this is a backwards incompatible change. If we had the SDK to design again, we probably wouldn't even have a default host:port, we'd _require_ it be set...

Added comment on #1706 mentioning that we don't want to automatically make workflow tasks with local activities start failing on poller stop (due to task timeout because we stop heartbeating),...

Yes, unfortunately due to internals and the generic nature of `env`, [`Value.Set`](https://pkg.go.dev/github.com/spf13/pflag#Value.Set) only sets the first value of a string collection. We do expect `env` to be deprecated and replaced...

@fairlydurable - sorry I missed your comment until now. I think https://github.com/temporalio/documentation/pull/3113/files#r1866332621 still needs to be addressed