You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ra...@apache.org on 2016/08/16 08:25:35 UTC
[1/2] git commit: updated refs/heads/master to d3e4551
Repository: cloudstack
Updated Branches:
refs/heads/master 56a352650 -> d3e45515e
updated contributin.md
As this is the doc that is showed while creating a pull request, updated it with information from release principles wiki.
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/4c387fee
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/4c387fee
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/4c387fee
Branch: refs/heads/master
Commit: 4c387feebbffdb6724fffa28f67c65fdb82bcdc2
Parents: 2875af7
Author: Rajani Karuturi <ka...@users.noreply.github.com>
Authored: Fri Aug 5 15:06:52 2016 +0530
Committer: GitHub <no...@github.com>
Committed: Fri Aug 5 15:06:52 2016 +0530
----------------------------------------------------------------------
CONTRIBUTING.md | 25 ++++++++++++++++++++++++-
1 file changed, 24 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/4c387fee/CONTRIBUTING.md
----------------------------------------------------------------------
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index c4ad197..6ac3ad5 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -3,10 +3,26 @@ Contributing to Apache CloudStack (ACS)
Summary
-------
-This document covers how to contribute to the ACS project. These instructions assume you have a GitHub.com account, so if you don't have one you will have to create one. Your proposed code changes will be published to your own fork of the ACS project and you will submit a Pull Request for your changes to be added.
+This document covers how to contribute to the ACS project. ACS uses github PRs to manage code contributions.
+These instructions assume you have a GitHub.com account, so if you don't have one you will have to create one. Your proposed code changes will be published to your own fork of the ACS project and you will submit a Pull Request for your changes to be added.
_Lets get started!!!_
+Bug fixes
+---------
+
+It's very important that we can easily track bug fix commits, so their hashes should remain the same in all branches.
+Therefore, a pull request (PR) that fixes a bug, should be sent against a release branch.
+This can be either the "current release" or the "previous release", depending on which ones are maintained.
+Since the goal is a stable master, bug fixes should be "merged forward" to the next branch in order: "previous release" -> "current release" -> master (in other words: old to new)
+
+Developing new features
+-----------------------
+
+Development should be done in a feature branch, branched off of master.
+Send a PR(steps below) to get it into master (2x LGTM applies).
+PR will only be merged when master is open, will be held otherwise until master is open again.
+No back porting / cherry-picking features to existing branches!
Fork the code
-------------
@@ -30,6 +46,7 @@ $ git rebase upstream/master
Making changes
--------------
+
It is important that you create a new branch to make changes on and that you do not change the `master` branch (other than to rebase in changes from `upstream/master`). In this example I will assume you will be making your changes to a branch called `feature_x`. This `feature_x` branch will be created on your local repository and will be pushed to your forked repository on GitHub. Once this branch is on your fork you will create a Pull Request for the changes to be added to the ACS project.
It is best practice to create a new branch each time you want to contribute to the project and only track the changes for that pull request in this branch.
@@ -72,6 +89,8 @@ Make a GitHub Pull Request to contribute your changes
When you are happy with your changes and you are ready to contribute them, you will create a Pull Request on GitHub to do so. This is done by pushing your local changes to your forked repository (default remote name is `origin`) and then initiating a pull request on GitHub.
+Please include JIRA id, detailed information about the bug/feature, what all tests are executed, how the reviewer can test this feature etc. Incase of UI PRs, a screenshot is preferred.
+
> **IMPORTANT:** Make sure you have rebased your `feature_x` branch to include the latest code from `upstream/master` _before_ you do this.
``` bash
@@ -105,3 +124,7 @@ $ git checkout master
$ git branch -D feature_x
$ git push origin :feature_x
```
+
+Release Principles
+------------------
+Detailed information about ACS release principles is available at https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+principles+for+Apache+CloudStack+4.6+and+up
[2/2] git commit: updated refs/heads/master to d3e4551
Posted by ra...@apache.org.
Merge pull request #1628 from karuturi/patch-1
updated contributing.mdAs this is the doc that is showed while creating a pull request, updated it with information from release principles wiki.
* pr/1628:
updated contributin.md
Signed-off-by: Rajani Karuturi <ra...@accelerite.com>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/d3e45515
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/d3e45515
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/d3e45515
Branch: refs/heads/master
Commit: d3e45515ee30642c657ee853bfa448290da54910
Parents: 56a3526 4c387fe
Author: Rajani Karuturi <ra...@accelerite.com>
Authored: Tue Aug 16 13:54:24 2016 +0530
Committer: Rajani Karuturi <ra...@accelerite.com>
Committed: Tue Aug 16 13:54:24 2016 +0530
----------------------------------------------------------------------
CONTRIBUTING.md | 25 ++++++++++++++++++++++++-
1 file changed, 24 insertions(+), 1 deletion(-)
----------------------------------------------------------------------