search

microsoft / cpp_client_telemetry

1DS C++ SDK
Apache License 2.0
85 stars 48 forks source link

Remove go wrapper #1268

Closed ThomsonTan closed 2 weeks ago

ThomsonTan commented 2 weeks ago

The go wrapper is not in complete shape. As we haven't heard about ask of this for years, I'd delete it from the repo now to avoid any maintenance burden for it.

We can always bring it back once we need it.

maxgolov commented 2 weeks ago

Yeah, it was used by BingAds. You can most likely remove it safely. I am not aware of other customers.

lalitb commented 2 weeks ago

The CI errors are not related to this PR, possibly related to the recent macOS runner image changes. We can address that separately, and force merge this PR.