Permalink
Switch branches/tags
Nothing to show
Find file Copy path
Fetching contributors…
Cannot retrieve contributors at this time
108 lines (73 sloc) 4.72 KB

Contribution

Any kind of contribution is encouraged, e.g., Jira items or patchsets.

LF ID Application

All the tools require an Linux Foundation (LF) ID.

If you do not have an LF ID, can apply one for free.

Jira board usage

We are using Jira to track the project progress, and welcome to report bug issues or create to-do tasks there. Each item should try keeping simple and focused, hence easy to fix and review.

After login with your LF ID, you can see those task items with one of the following statuses:

  • To Do: Available for picking and fix.
  • In Progress: Some on already picked it (check the assignee) to work on.
  • Under Review: Related patchset has been submitted for review, and added as comment under the Jira item.
  • Done: Patchset merged, the item has been resolved.

In brief, if you want to contribute, create or find some To Do item, and assign it to yourself, then update its status to In Progress. After the item is fixed, remember to mark it as Under Review and Done when the patch is submitted and merged.

Questions and discussions

  • RocketChat: technical discussions and questions, login with your LFID.

Code Commit Steps

The project employs Gerrit as the code commit/review system.

*Before committing code, please go to Jira to create a new task or check if there's related existing one, then assign yourself as the assignee. Notice each task will get a Jira number like CE-26.

  • Clone the project to your working directory with your LFID.
$ git clone ssh://LFID@gerrit.hyperledger.org:29418/cello && scp -p -P 29418 LFID@gerrit.hyperledger.org:hooks/commit-msg cello/.git/hooks/

(Optionally) Config your git name and email if not setup previously.

$ git config user.name "your name"
$ git config user.email "your email"

(Optionally) Setup git-review by inputting your LFID. Notice this is only necessary once.

$ git review -s
  • Assign yourself a To Do Jira task, mark it as In progress, then create a branch with the Jira task number off of your cloned repository, e.g., for CE-26, it can be:
$ cd cello
$ git checkout -b CE-26
  • After modifying the code, run make check to make sure all the checking is passed. Then Commit your code with -s to sign-off, and -a to automatically add changes (or run git add . to include all changes manually).
$ make check
  ...
  py27: commands succeeded
  py30: commands succeeded
  py35: commands succeeded
  flake8: commands succeeded
  congratulations :)

$ git commit -s -a

Example commit msg may look like (take CE-1234 for example):

[CE-1234] A short description of your change with no period at the end

You can add more details here in several paragraphs, but please keep each line
width less than 80 characters. A bug fix should include the issue number.

CE-1234 #done.

Change-Id: If2e142ea1a21bc4b42f702f9a27d70d31edff20d
Signed-off-by: Your Name <committer@email.address>
  • Submit your commit using git review, and mark the corresponding Jira item as Under Review.
$ git review
remote: Processing changes: new: 1, refs: 1, done
remote:
remote: New Changes:
remote:   http://gerrit.hyperledger.org/r/7915 [CE-26] Update the contribution documentation
remote:
To ssh://gerrit.hyperledger.org:29418/cello
 * [new branch]      HEAD -> refs/publish/master/CE-26
Switched to branch 'master'
Your branch is up-to-date with 'origin/master'.

Notice you will get a gerrit url like http://gerrit.hyperledger.org/r/7915, open it and check the status.

After the ci checking passed, add reviewers to the reviewer list and also post the gerrit url at the RocketChat channel. The patch will be merged into the master branch after passing the review, then mark the Jira item as Done.

  • If you need to refine the patch further as the reviewers may suggest, you can change on the same branch, and commit the new code with git commit -a --amend, and then use the git review command again.

Creative Commons License
This work is licensed under a Creative Commons Attribution 4.0 International License.