-
Notifications
You must be signed in to change notification settings - Fork 84
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
Link to examples of high quality pull requests in the pull request template #756
Comments
@mfisher87 Hey, do you have specific opinions on how we can approach this? I Think for solving this we need to have some PR'S that follow rules form PR template and we'll then just have to link to those PR'S. |
The hard part is finding those PRs :) I don't know that we'll find one "perfect PR" that ticks all the boxes easily, but maybe we can find PRs that exemplify specific things from the list? For example, we have this checklist item:
Instead of |
I will try to come up with a PR that follows the rubric. I think if we are able to encompass all the rules of the rubric in one single PR that would be great. I know it's the ideal case and it may not be plausible at this point in time but I think we should try! |
That sounds great! Thanks for being willing to dive deep on this! ❤️ |
@mfisher87 Can we use the PR for backwards compatibility as an Ideal PR? |
I think so! Based on this comment, which of our PR checklist items do you feel that PR sets a good example for? |
I feel my PR has proper methodology, what I mean by that is that you will have to make changes to CHANGELOG before making a PR which I guess is something that people contributing for the first time might look over. Also I feel I could have done better with some of the titles of my commits but I think they are not too bad(they are descriptive enough). These are some of things I can remember at this point in time. |
That sounds reasonable! Would you be willing to open a PR to modify our PR template? I'm thinking maybe we could add the example PR links as sub-bullets of the checklist items e.g. earthaccess/.github/pull_request_template.md Lines 35 to 37 in bb553e1
|
For example, #278 does a great job of providing a complete usage example, and our PR template can guide others to do the same!
The text was updated successfully, but these errors were encountered: