-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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: improve contributor guide to explain how to work with tickets #8999
Conversation
Thanks for expanding the contributor guide @alamb, very clear! |
|
||
You can find a curated | ||
[good-first-issue](https://github.com/apache/arrow-datafusion/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[good-first-issue](https://github.com/apache/arrow-datafusion/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22) |
Moved.
Co-authored-by: Jonah Gao <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @alamb would be nice to have a link to contribution.md from the top level page in
https://github.com/apache/arrow-datafusion#contributing-to-datafusion section
Excellent call -- done in 94bbca7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice thanks @alamb
Which issue does this PR close?
Part of #7013
Rationale for this change
Recently the question of who is / could be working on what ticket came up in discussions. For example"
POSITION
function #8969 (comment) from @Lordworms about what tickets were availableI think it would be good to clarify the contributing guide in this case
What changes are included in this PR?
Are these changes tested?
Docs only
Are there any user-facing changes?