The Code of Conduct explains the bare minimum behavior expectations the Workshopper organization requires of its contributors. Please read it before participating.
When opening new issues or commenting on existing issues on this repository please make sure discussions are related to concrete technical issues with the workshoppers.
For general help with learning via workshoppers, please file an issue at the NodeSchool discussion repository.
The workshopper organization has an open governance model and welcomes new contributors. Individuals making significant and valuable contributions are given commit-access to the project to contribute as they see fit. This project is more like an open wiki than a standard guarded open source project.
There are a few basic ground-rules for contributors (from openopensource.org):
- No
--force
pushes or modifying the Git history in any way. - Non-master branches ought to be used for ongoing work.
- External API changes and significant modifications ought to be subject to an internal pull-request to solicit feedback from other contributors.
- Internal pull-requests to solicit feedback are encouraged for any other non-trivial contribution but left to the discretion of the contributor.
- Contributors should attempt to adhere to the prevailing code style.
Declaring formal releases remains the prerogative of the project maintainer.
This is an experiment and feedback is welcome! This document may also be subject to pull-requests or changes by contributors where you believe you have something valuable to add or change.
This document will guide you through the contribution process.
Fork the workshopper and check out your copy locally.
In the examples below, replace
<username>
with your username, and<workshopper>
with the name of the workshopper.
$ git clone git@github.com:<username>/<workshopper>.git
$ cd <workshopper>
$ git remote add upstream git://github.com/workshopper/<workshopper>.git
For developing new features and bug fixes, the master
branch should be pulled
and built upon.
In case of doubt, open an issue in the specific workshopper's issues section or contact one of the maintainers listed in that workshopper's README. Especially do so if you plan to work on something big. Nothing is more frustrating than seeing your hard work go to waste because your vision does not align with the project team. We have two main Gitter channels, nodeschool/organizers for general help and questions, and nodeschool/workshoppers for development of NodeSchool workshoppers specifically.
Create a branch and start hacking:
$ git checkout -b my-branch -t origin/master
Make sure git knows your name and email address:
$ git config --global user.name "J. Random User"
$ git config --global user.email "j.random.user@example.com"
Writing good commit logs is important. A commit log should describe what changed and why. Follow these guidelines when writing one:
- The first line should be a short description of the change. All words in the description should be in lowercase with the exception of proper nouns, acronyms, and the ones that refer to code, like function/variable names.
- Keep the second line blank.
- Follow with a longer description of your changes.
A good commit log can look something like this:
explain the commit in one line
Body of commit message is a few lines of text, explaining things
in more detail, possibly giving some background about the issue
being fixed, etc. etc.
The body of the commit message can be several paragraphs, with
line breaks as necessary.
The header line should be meaningful; it is what other people see when they
run git shortlog
or git log --oneline
.
If your patch fixes an open issue, you can add a reference to it at the end
of the log. Use the Fixes:
prefix and the full issue URL. For example:
Fixes: https://github.com/workshopper/learnyounode/issues/1337
Use git rebase
(not git merge
) to sync your work from time to time.
$ git fetch upstream
$ git rebase upstream/master
Bug fixes and features should come with tests. Please note that not all workshoppers currently have tests. This could make a great contribution! See issue #13 for more information
$ git push origin my-branch
Go to https://github.com/<username>/<workshopper>
and select your branch.
Click the 'Pull Request' button and fill out the form.
Pull requests are usually reviewed within a few days. If there are comments to address, apply your changes in a separate commit and push that to your branch. Post a comment in the pull request afterwards; GitHub does not send out notifications when you add commits.
By making a contribution to this project, I certify that:
-
(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or
-
(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or
-
(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.
-
(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.