You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, as suggested by @pepone the BuildTelemetry works via a shared custom build task that gets called by the IceRpc.Slice.Tools and IceRpc.Protobuf.Tools. I think this approach is good for now as it is simple and does the job. However in the future, we should refactor this logic into a protoc plugin or slicec plugin.
The text was updated successfully, but these errors were encountered:
Somewhat related, but could be split into a different issue, is @bernardnormier suggested an intermediate refactor before this too where we would split the task into a separate protobuf task and slicec task and have the build telemetry client logic live in these tasks rather than being a separate executable that gets invoked via the task.
I think given the eventual shift to a plugin this intermediate refactor is not really necessary.
Currently, as suggested by @pepone the BuildTelemetry works via a shared custom build task that gets called by the
IceRpc.Slice.Tools
andIceRpc.Protobuf.Tools
. I think this approach is good for now as it is simple and does the job. However in the future, we should refactor this logic into aprotoc
plugin orslicec
plugin.The text was updated successfully, but these errors were encountered: