cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bhais...@apache.org
Subject [40/50] [abbrv] git commit: updated refs/heads/4.9-bountycastle-daan to 8a03127
Date Thu, 18 Aug 2016 10:02:24 GMT
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/4.9-bountycastle-daan
Commit: 4c387feebbffdb6724fffa28f67c65fdb82bcdc2
Parents: 2875af7
Author: Rajani Karuturi <karuturi@users.noreply.github.com>
Authored: Fri Aug 5 15:06:52 2016 +0530
Committer: GitHub <noreply@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



Mime
View raw message