neilconway commented on code in PR #24510: URL: https://github.com/apache/datafusion/pull/24510#discussion_r4173294050
########## docs/source/contributor-guide/index.md: ########## @@ -54,24 +54,29 @@ Contributors drive the project forward based on their own priorities and interests and thus you are free to work on any issue that interests you. If someone is already working on an issue that you want or need but hasn't -been able to finish it yet, you should feel free to work on it as well. In -general it is both polite and will help avoid unnecessary duplication of work if -you leave a note on an issue when you start working on it. +been able to finish it yet, feel free to help them out. -If you want to work on an issue which is not already assigned to someone else -and there are no comment indicating that someone is already working on that -issue then you can assign the issue to yourself by submitting a single word -comment `take`. This will assign the issue to yourself. However, if you are -unable to make progress you should unassign the issue by commenting a single -word `untake`. +If there is an existing PR for an issue you plan to work on, please review that +PR before opening a new one. Duplicate, unacknowledged PRs consume valuable Review Comment: Should we say "if there is an existing open PR"? There's a lot of older issues with one or more stale, long-closed PRs from prior attempts to work on them. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
