From 4d2b5e93147416f88a1bd593aac76b38b1d02090 Mon Sep 17 00:00:00 2001 From: Mrugesh Mohapatra <1884376+raisedadead@users.noreply.github.com> Date: Wed, 5 Oct 2022 19:38:50 +0530 Subject: [PATCH] docs: add newbie guideline and template (#47834) * docs: add newbie guideline and template * fix: open the sidebar by default * chore: apply review * Apply suggestions from code review Co-authored-by: Naomi Carrigan * Update docs/how-to-open-a-pull-request.md Co-authored-by: Ilenia * Update docs/how-to-open-a-pull-request.md Co-authored-by: Tom <20648924+moT01@users.noreply.github.com> * Update docs/moderator-handbook.md Co-authored-by: Tom <20648924+moT01@users.noreply.github.com> Co-authored-by: Naomi Carrigan Co-authored-by: Ilenia Co-authored-by: Tom <20648924+moT01@users.noreply.github.com> --- docs/how-to-open-a-pull-request.md | 22 +++++++++++++++++++++- docs/index.html | 2 +- docs/moderator-handbook.md | 20 ++++++++++++++++++++ 3 files changed, 42 insertions(+), 2 deletions(-) diff --git a/docs/how-to-open-a-pull-request.md b/docs/how-to-open-a-pull-request.md index c73f209c7ba..d857f8b1563 100644 --- a/docs/how-to-open-a-pull-request.md +++ b/docs/how-to-open-a-pull-request.md @@ -2,8 +2,18 @@ A pull request (PR) enables you to send changes from your fork on GitHub to freeCodeCamp.org's main repository. Once you are done making changes to the code, you can follow these guidelines to open a PR. +We expect our contributors to be aware of the process specific to this project. Following the guidelines religiously earns you the respect of fellow maintainers and saves everyone time. + +Some examples of this are: + +1. Do not edit files directly through GitHub – while you can, it's not a good idea. +2. Make sure you follow the PR checklist and not just tick things off; otherwise, we won't take you seriously. +3. Use the correct way to link issues in the description of the PR by updating the `XXXXXX`. Do not just add issue numbers everywhere and anywhere you feel like. +4. Keep mentions and review requests to a minimum. We understand you are excited about contributing, and our maintainers will get back to you as soon as they get a chance. +5. Do not work directly off your `main` branch - create a new branch for the changes you are working on. + > [!NOTE] -> Your PR should be in English. See [here](index.md#translations) for how to contribute translations. +> Your PR should be targeting changes to the English curriculum only. Read [this guide](index.md#translations) instead for contributing to translations. ## Prepare a good PR title @@ -45,14 +55,24 @@ Some examples of good PR titles would be: 1. Once the edits have been committed, you will be prompted to create a pull request on your fork's GitHub Page. +
+ See screenshot + ![Image - Compare & pull request prompt on GitHub](https://contribute.freecodecamp.org/images/github/compare-pull-request-prompt.png) +
+ 2. By default, all pull requests should be against the freeCodeCamp main repo, `main` branch. Make sure that your Base Fork is set to freeCodeCamp/freeCodeCamp when raising a Pull Request. +
+ See screenshot + ![Image - Comparing forks when making a pull request](https://contribute.freecodecamp.org/images/github/comparing-forks-for-pull-request.png) +
+ 3. Submit the pull request from your branch to freeCodeCamp's `main` branch. 4. Include a more detailed summary of the changes you made and how your changes are helpful in the body of your PR. diff --git a/docs/index.html b/docs/index.html index e2ea7dc1d32..3f2f69a5047 100644 --- a/docs/index.html +++ b/docs/index.html @@ -53,7 +53,7 @@ - +
diff --git a/docs/moderator-handbook.md b/docs/moderator-handbook.md index 4d63dea4960..05f0a47a8ac 100644 --- a/docs/moderator-handbook.md +++ b/docs/moderator-handbook.md @@ -488,6 +488,26 @@ We are closing this pull request. Please suggest links and other details to add If you think we're wrong in closing this issue, please request for it to be reopened and add further clarification. Thank you and happy coding. ``` +### Adding comment about newbie mistakes + +```markdown +As a new contributor, we encourage you to read our [contributing guidelines](https://contribute.freecodecamp.org). + +We expect our contributors to be aware of the process specific to this project. Following the guidelines religiously earns you the respect of fellow maintainers and saves everyone time. + +Some examples of this are: + +1. Do not edit files directly through GitHub – while you can, it's not a good idea. +2. Make sure you follow the PR checklist and not just tick things off; otherwise, we won't take you seriously. +3. Use the correct way to link issues in the description of the PR by updating the `XXXXXX`. Do not just add issue numbers everywhere and anywhere you feel like. +4. Keep mentions and review requests to a minimum. We understand you are excited about contributing, and our maintainers will get back to you as soon as they get a chance. +5. Do not work directly off your `main` branch - create a new branch for the changes you are working on. + +We appreciate you taking the time to help us, and we hope to see more contributions from you. + +Happy Contributing. +``` + ### Closing Invalid Issues > When an issue relates to the camper's code.