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

update the tikv community meeting #7382

Closed
wants to merge 2 commits into from
Closed

update the tikv community meeting #7382

wants to merge 2 commits into from

Conversation

WT-Liu
Copy link
Contributor

@WT-Liu WT-Liu commented Apr 8, 2020

Signed-off-by: Wenting Liu liuwenting@pingcap.com

Signed-off-by: Wenting Liu <liuwenting@pingcap.com>
@sre-bot sre-bot added the contribution Type: PR - From contributors label Apr 8, 2020
@WT-Liu
Copy link
Contributor Author

WT-Liu commented Apr 8, 2020

@dcalvin PTAL and thanks

Copy link
Member

@dcalvin dcalvin left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -118,6 +118,7 @@ Quick links:

* [Meeting notes](https://docs.google.com/document/d/1CWUAkBrcm9KPclAu8fWHZzByZ0yhsQdRggnEdqtRMQ8/edit)
* [Zoom meeting link](https://zoom.us/my/cncftikvproject)
* [Add to my calendar](https://calendar.google.com/calendar/r/month/2020/4/23?eid=MXZiaDg4YmEwYW1rbDlwajRtdjIyb2w3ZTlfMjAyMDA0MjNUMDEwMDAwWiBsaW51eGZvdW5kYXRpb24ub3JnX281YXZqbHZ0MmNhZTlicTdhOTVlbWM0NzQwQGc&ctz=America/New_York&sf=true)
Copy link
Member

Choose a reason for hiding this comment

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

Can the link be a regular one instead of editing from time to time?

Copy link
Member

Choose a reason for hiding this comment

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

This is generated from the CNCF. And also it makes more sense to add to calendar only for the current week because we can't make the decision on behalf of other people.

Copy link
Member

Choose a reason for hiding this comment

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

Then I don't think it's necessary to put the scheduling information in README, maybe we need to put it on a web page.

@dcalvin
Copy link
Member

dcalvin commented Apr 15, 2020

/merge

@sre-bot sre-bot added the status/can-merge Status: Can merge to base branch label Apr 15, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 15, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Apr 15, 2020

@WT-Liu merge failed.

@WT-Liu WT-Liu closed this May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution Type: PR - From contributors status/can-merge Status: Can merge to base branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants