You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@dubbo.apache.org by GitBox <gi...@apache.org> on 2021/09/13 03:01:43 UTC

[GitHub] [dubbo-go-samples] zouyx commented on a change in pull request #243: add dubbogo tracing sample

zouyx commented on a change in pull request #243:
URL: https://github.com/apache/dubbo-go-samples/pull/243#discussion_r706972467



##########
File path: tracing/dubbo/go-client/cmd/client.go
##########
@@ -20,66 +20,48 @@ package main
 import (
 	"context"
 	"fmt"
+	"github.com/opentracing/opentracing-go"

Review comment:
       format this import

##########
File path: tracing/dubbo/go-server/cmd/server.go
##########
@@ -19,31 +19,23 @@ package main
 
 import (
 	"fmt"
+	"github.com/opentracing/opentracing-go"

Review comment:
       as above

##########
File path: tracing/dubbo/go-server/pkg/user.go
##########
@@ -19,49 +19,43 @@ package pkg
 
 import (
 	"context"
+	"fmt"
+	gxlog "github.com/dubbogo/gost/log"

Review comment:
       > 我reformat代码后,没啥变化,这个是哪里不合格式规范吗?
   
   refer this rule : https://github.com/apache/dubbo-go/blob/master/CONTRIBUTING.md#34-import




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org