Skip to content
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

docs:add README.md #3

Merged
merged 8 commits into from
Nov 20, 2023
Merged

Conversation

ViolaPioggia
Copy link
Collaborator

What type of PR is this?

docs

Check the PR title.

docs:add README.md

(Optional) Translate the PR title into Chinese.

增加了 README 文档

(Optional) Which issue(s) this PR fixes:

Fixes cloudwego/kitex#1140

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README_CN.md Outdated Show resolved Hide resolved
README_CN.md Outdated Show resolved Hide resolved
README_CN.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@whalecold
Copy link
Member

/lgtm

[中文](./README_CN.md)

etcd as config center for service governance.

Choose a reason for hiding this comment

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

参考 https://github.com/hertz-contrib/registry/tree/main/licenses 补充一下直接依赖的 license

config-nacos 辛苦有时间也补充一下 @whalecold

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

@li-jin-gou
Copy link

li-jin-gou commented Nov 20, 2023

重新 format 一下 import 的格式吧
image

https://github.com/ViolaPioggia/config-etcd/blob/config-etcd/etcd/etcd.go#L25 && 不只这一处

Copy link

New changes are detected. LGTM label has been removed.

Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: li-jin-gou, ViolaPioggia

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@li-jin-gou li-jin-gou merged commit 4ac6aec into kitex-contrib:main Nov 20, 2023
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposal: 对接 etcd 配置中心 | etcd As Config Center
5 participants