This repository has been archived by the owner on Aug 20, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 78
Home
kalyan kondapally edited this page Sep 15, 2013
·
37 revisions
We are on IRC: #ozone-wayland
Instructions to set up development environment can be found in ReadMe file.
Get familiar with Git Hub. Useful pages:
We use GitHub's Pull Request mechanism to accept changes.
Coding Style:
- [Coding style is the same as chromium] (http://www.chromium.org/developers/coding-style)
Commit-Message:
- First line should be summary not exceeding 50 characters.
- Second line should be a blank line.
- Than an Explanatory text with lines not exceeding 72 characters.
Running Tests:
TODO
Submitting Pull Requests:
- Start a new issue.
- Find an OWNER and discuss whether the issue is valid. Ping us on IRC.
- Branch repository, work on solution.
- When ready for review, advertise the branch to other project members.
- Get review from an OWNER, meaning an explicit LGTM.
- It's quite common that initial solution is changed during review process. Separate final solution to semantically related commits and send pull request. Wait for owner to manage the merge.
- Unless critical fix, nothing should go into master branch without review from one other OWNER.
- Ensure that latest code always builds.
- Ensure changes don't introduce any new regression and merge pull requests. Hopefully, we are able to soon run chromium tests. This should reduce the burden.
- Guarantee integrity of whole project together with other owners.
Landing Pull Requests:
TODO
Rebasing process:
TODO