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

Improve readme #88

Merged
merged 3 commits into from
Apr 20, 2022
Merged

Conversation

hormes
Copy link
Member

@hormes hormes commented Apr 20, 2022

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it

Ⅳ. Special notes for reviews

Signed-off-by: Fansong Zeng <fanster.z@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Apr 20, 2022

Codecov Report

Merging #88 (db329e9) into main (0a833f4) will not change coverage.
The diff coverage is n/a.

❗ Current head db329e9 differs from pull request most recent head ff64faf. Consider uploading reports for the commit ff64faf to get more accurate results

@@           Coverage Diff           @@
##             main      #88   +/-   ##
=======================================
  Coverage   40.81%   40.81%           
=======================================
  Files          66       66           
  Lines        5145     5145           
=======================================
  Hits         2100     2100           
  Misses       2795     2795           
  Partials      250      250           
Flag Coverage Δ
unittests 40.81% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0a833f4...ff64faf. Read the comment docs.

@eahydra
Copy link
Member

eahydra commented Apr 20, 2022

/lgtm

README.md Outdated
@@ -57,7 +57,7 @@ our [Community Membership](docs/community/community-membership.md) page.
You can reach the maintainers of this project at:

- [Slack](https://join.slack.com/t/koordinator-sh/shared_invite/zt-1756qoub4-Cn4~esfdlfAPsD7cwO2NzA)
- DingTalk: Search Group ID `33383887` or scan the following QR Code
- DingTalk(Chinese): Search Group ID `33383887` or scan the following QR Code

<div>
<img src="https://github.com/koordinator-sh/koordinator/raw/main/docs/images/dingtalk.jpg" width="300" alt="Dingtalk QRCode">
Copy link
Member

Choose a reason for hiding this comment

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

You need to change the url of qrcode to dingtalk.png.

p.s. 我个人认为原本包含群组名称的图片会比较好看,可以用小工具把人数给 P 掉~

Copy link
Member Author

Choose a reason for hiding this comment

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

改了一下

@koordinator-bot koordinator-bot bot removed the lgtm label Apr 20, 2022
Signed-off-by: Fansong Zeng <fanster.z@gmail.com>
@zwzhang0107
Copy link
Contributor

/lgtm

Copy link
Member

@jasonliu747 jasonliu747 left a comment

Choose a reason for hiding this comment

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

/lgtm

@hormes
Copy link
Member Author

hormes commented Apr 20, 2022

/approve

@koordinator-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hormes, jasonliu747

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

The pull request process is described 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

@koordinator-bot koordinator-bot bot merged commit 1dbd7c8 into koordinator-sh:main Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants