Skip to content

Add go.mod file. #38

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.

Sign up for GitHub

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

Jump to bottom
Open
radioinmyhead wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add go.mod file. #38

radioinmyhead wants to merge 2 commits into from

Conversation

@radioinmyhead
Copy link
Contributor

@radioinmyhead radioinmyhead commented Mar 13, 2020 •
edited
Loading

  • 添加go.mod，兼容那些使用mod管理的小伙伴
  • 修改Tag结构体，减少冗余：之前的用法一般是：model.TagModel.TagKey 改了之后就可以是：model.Tag.Key。之前的写法明显不科学。而且之前的很多TagModel 都是通用的。不用单独定义一次
  • 为CreateInstanceArgs等多个结构体添加json tag，否则后端server不认参数
  • 为各种args提取一个简单的meta信息，同样为result提取一个meta信息

gola
gola reviewed Mar 16, 2020
View reviewed changes
go.mod
@@ -0,0 +1,3 @@
module github.com/baidubce/bce-sdk-go
Copy link
Member

@gola gola Mar 16, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

关于此SDK的使用情况及进展，能否给一个联系方式沟通一下？我的联系方式[email protected]

Copy link
Contributor Author

@radioinmyhead radioinmyhead Mar 16, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

用Hi联系吧，我们做个小群

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gola gola gola left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@radioinmyhead @gola