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.

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

docs: Feel relax in community and don't hurry #1987

Merged
merged 4 commits into from
Apr 14, 2023
Merged

docs: Feel relax in community and don't hurry #1987

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
6446a64
docs: Relax in community and don't hurry
Xuanwo Apr 14, 2023
59a911c
90 days!
Xuanwo Apr 14, 2023
7ff1c96
Don't enforece to close PRs
Xuanwo Apr 14, 2023
a08136b
Make it more clear
Xuanwo Apr 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 3 additions & 9 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ Hello, welcome to the OpenDAL Community! This document will guide you on how to

## Taking Issues

**OpenDAL's Issue Roundtrip: 14 days**

It is highly recommended to request an assignment before beginning work on any issues to avoid conflicts with other community members.

It is encouraged to post your work progress in the issues comment so that the community can stay informed about ongoing developments and offer any necessary assistance.
Expand All @@ -16,12 +14,8 @@ By removing the assignment, we create space for other community members to take

## Sending PRs

**OpenDAL's PR Roundtrip: 24 hours**

OpenDAL promises fast iteration (I mean very fast).

We commit to reviewing your PRs within 24 hours, and often within 8 hours. In return, we expect you to respond to our review within 24 hours as well. *It doesn't mean you MUST respond to us within 24 hours. You can contribute at your own pace.*
We will try our best to review your PRs within 24 hours, and often within 8 hours.

If a PR receives no response for 24 hours, it will be closed. Please note that closing a PR does not mean rejection; we simply aim to keep our community iterating quickly by only tracking PRs awaiting review. By closing on-going (instead of on-reviewing) PR allow both of us foucing our own work.
If a PR does not receive a response within 24 hours, it will be converted to draft status. To indicate that you are ready for review, please mark the PR as `ready for review`. By default, all reviewers ignore draft PRs.
Xuanwo marked this conversation as resolved.
Show resolved Hide resolved

You are welcome to reopen the PR at any time when you are ready for a new review.
If a PR remains inactive for 90 days, it will be closed. It is important to note that closing a PR does not necessarily mean rejection; our goal is to keep our community iterating quickly. You are welcome to reopen the PR at any time when you are ready for another review.