Yanick Champoux > Dancer-1.3126 > Dancer::Development::Integration

Download:
Dancer-1.3126.tar.gz

Annotate this POD

Website

View/Report Bugs
Source   Latest Release: Dancer-1.3130

NAME ^

Dancer::Development::Integration - guide for Dancer's core-team members

DESCRIPTION ^

This documentation describes the procedure used for integrators to review and merge contributions sent via pull-requests.

Every core-team member should read and apply the procedures described here. This will allow for a better history and more consistency in our ways of handling the (increasing number!) of pull requests.

TERMS ^

We will first define the most important terms used in this documentation:

PROCEDURES ^

Processing a Pull Request

This procedure describes how an integrator should process a PR.

Let's say the user $user has sent a PR, he has followed the instructions described in Dancer::Development so his work is based on the integration branch (devel).

All the procedure described here is designed to avoid unnecessary recursive-merge, in order to keep a clean and flat history in the integration branch.

Of course, we could just pull from $user into our devel branch, but this would shift the history because of recursive merge, most of the time.

To avoid that, we're going to pull the commits of $user into a temporary branch, and then cherry-pick the commits we want.

In order to have a clean history, like the one we got with git-flow when working on a feature, we're going to do that in a topic branch, named review/$user. Then, this branch will be merged into devel and we will just have to drop it.

First, we make sure we are in sync with origin/devel

    git checkout devel
    git pull origin devel 

Then, from that branch we create a temp sandbox

    git checkout -b temp

We pull here from $user

    git pull <user repo> <pr/branch>

Here, either the pull was run as a fast-forward or as a recursive merge. If we have a FF, we can forget about the temp branch and do the pull directly in devel. If not, we'll have to cherry-pick the commits by hand.

From devel, we first create the final review branch:

    git checkout devel
    git checkout -b review/$user

Then we cherry-pick all the commits we want. To know them, we just have to go into temp and inspect the history (with git log).

When we have the list of commits we want:

    for commit in C1 C2 C3 ... CN
    do
        git cherry-pick $commit
    done

(Another option is to use git rebase -i to manually select the list of commits to cherry-pick/rebase.)

Then we can review the code, do whatever we want, maybe add some commits to change something.

When we're happy with the change set, we can merge into devel:

    git checkout devel
    git merge --no-ff review/$user

Note the --no-ff switch is used to make sure we'll see a nice commit named Merge branch 'review/$user' into devel. This is on purpose and mimic the behaviour of git-flow.

Your local devel branch is now merged, and can be pushed to the remote.

    $ git push origin devel

RELEASE CYCLES ^

We have one main release cycle. This is the release cycle based on the devel branch. We use this branch to build new releases, with new stuff all the new shiny commits we want.

Those release are built with git-flow (with git-flow release) and are then uploaded to CPAN.

Since Dancer 1.2, we also have another parallel release cycle which is what we call the frozen branch. It's a maintenance-only release cycle. That branch is created from the tag of the first release of a stable version (namely a release series with an even minor number).

This branch must be used only for bug-fixing the stable releases. Nothing new should occur in that branch.

Let's take an example with Dancer 1.2003 and Dancer 1.3002.

AUTHOR ^

This documentation has been written by Alexis Sukrieh <sukria@sukria.net>.

syntax highlighting: