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][COMMUNITY] Committer guide and tips #2468

Merged
merged 1 commit into from
Jan 24, 2019
Merged

Conversation

tqchen
Copy link
Member

@tqchen tqchen commented Jan 19, 2019

Summarize some tips and lessons for committers

@tqchen
Copy link
Member Author

tqchen commented Jan 19, 2019

cc @dmlc/tvm-comitter

Working on an open source project can be rewarding,
but also be a bit overwhelming sometimes.
A little bit of time management might be helpful to alleviate the problem.
For example, some committers have a "community day" in a week
Copy link
Member

Choose a reason for hiding this comment

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

shall we make the "community day" record public so that people can know whom to reach out? only for committers who are willing to have an assigned "community day" of course.

Also we better find a way to enable committers/reviewers report their out-of-office status. e.g., github's "set your status" might be useful.

Copy link
Member

Choose a reason for hiding this comment

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

"community day" record public

A pinned post on the forum would be great, but people might not know to check it. Maybe putting a blurb in the ISSUE_TEMPLATE would help.

github's "set your status" might be useful

👍 just depends on whether you think that's an intrusion into one's privacy :)

@tqchen tqchen merged commit 65548f9 into apache:master Jan 24, 2019
@tqchen
Copy link
Member Author

tqchen commented Jan 24, 2019

Thanks, @yzhliu @nhynes for the helpful feedbacks, this is now merged.

Anthony-Mai pushed a commit to Anthony-Mai/tvm that referenced this pull request Jan 25, 2019
zhiics pushed a commit to zhiics/tvm that referenced this pull request Jan 30, 2019
@ZihengJiang ZihengJiang mentioned this pull request Feb 1, 2019
merrymercy pushed a commit to merrymercy/tvm that referenced this pull request Feb 18, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants