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.

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

Notes to contributors #6

Open
andrealaw opened this issue Sep 11, 2017 · 4 comments
Open

Notes to contributors #6

andrealaw opened this issue Sep 11, 2017 · 4 comments
Labels

Comments

@andrealaw
Copy link
Collaborator

andrealaw commented Sep 11, 2017

General guidelines on how to contribute

Pick an issue

  1. Find an item that you can contribute to and pick it up by leaving your name under the issue so others will know that someone has picked up the task.
  2. Request for a new branch if that is not available (optional).
  3. Submit a pull request and ask for a review.
  4. Once the code changes get reviewed and tested, they will be merged to the respective branch for production.
  5. Your pull request is successful and you will have this contribution shown in your Github account.

Add a new issue

If you want to add a new issue or discuss about a particular change for the website, like building a new template, adding a new section, or migrating to a new platform, please add a new issue and assign a relevant label to it.

Review, merge and delete merged branches

  1. Contact the administration team if you can be a reviewer.
  2. Reviewers should test the code changes before merging to the respective branch.
  3. Leave your comment if further changes are required for the pull request.
  4. Before pressing the merge button, check the commit message. If you want to close an issue at the same time, follow this pattern to ensure keywords(e.g., close or resolve) are used and an issue number is added (e.g., Link doesn't work #9) for tracking.
  5. Delete the branch according to the Github instructions.

About the administrators

This repository is currently maintained by a team of volunteers from Free Code Camp Hong Kong. You can find a list of members here: @FreeCodeCampHongKong/admin

Thanks!

@anglyeds
Copy link

anglyeds commented Sep 16, 2017

Thank for the notes, I just noticed there are few codes that I have to review.

@gregorywong
Copy link
Member

@anglyeds So actually, Isaac has added a new plugin which allows the pull requests to be previewed, meaning that you don't have to fetch the code and open it locally. You can just click the green check mark next to the commit hash to get a preview.

See this comment for more information: #21 (comment)

@gregorywong
Copy link
Member

@anglyeds Clarification: It's better to click on "Details" to see the latest commit:

screen shot 2017-09-16 at 11 08 49 pm

@gregorywong
Copy link
Member

Should I follow this guide and consolidate our contributor's guidelines in one place according to this guide?

https://help.github.com/articles/setting-guidelines-for-repository-contributors/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants