- 
                Notifications
    
You must be signed in to change notification settings  - Fork 21
 
change service.proto to polaris_service.proto #137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
| @@ -1,15 +1,16 @@ | |||
| module github.com/polarismesh/specification | |||
| module github.com/lendsee/polarismesh-specification | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
为什么这个要改动
| 
           测试过兼容性吗  | 
    
          
 这确实没有,不过我的改动很简单,不影响代码,以下是我的改动步骤, 可以看一下 
  | 
    
          
 另外,你近期在main上提交了两个改动, 应该是没改全,所以使用 bash build.sh 生成代码时,会生成一些多余文件,导致代码跑不通,我以上的改动是基于 v1.5.5-alpha.1 的变更。  | 
    
          
 @lendsee 具体是哪两个依赖,方便提供一个例子吗?  | 
    
          
 package main import ( func main(){ go mod tidy 可以成功。  | 
    
| 
           两个包的package是不一样的,是否可以规避这个问题呢?  | 
    
          
 主分支更新了,不会出现这个问题了。  | 
    
rocketmq和polaris共同引用时,会报service.proto冲突,修改文件名,并重新变异