Diff for "Patches"

Not logged in - Log In / Register
Differences between revisions 2 and 9 (spanning 7 versions)
⇤ ← Revision 2 as of 2009-04-10 15:29:41 →
Size: 1799
Editor: kfogel
Comment: Formatting tweak.
← Revision 9 as of 2009-08-05 14:24:24 → ⇥
Size: 27
Editor: kfogel
Comment:
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
#format wiki
#language en
#pragma section-numbers off

= Submitting a Patch to Launchpad =

This page covers the process of getting a code change into Launchpad. You should already have your Launchpad sources (see [[https://wiki.canonical.com/Launchpad/RocketFuelSetup| Rocketfuel Setup]]) and be familiar with the [[StyleGuides|coding style guidelines]].

 1. Make sure your change is about one single thing (say, a specific bugfix), and that all the revisions for it are committed on your Bazaar branch.
 1. Push the branch up to your Launchpad user account: '''{{{bzr push my_bugfix123_branch lp:~my_username/launchpad/bugfix123}}}'''
 1. [[Testing|Run tests]] on your branch. If the tests don't pass, go back to hacking, commit new changes, re-push, and re-test. Repeat until the branch passes.
 1. Please send in your [[ContributorAgreement|contributor agreement form]] now (it's an easy email process). We can't accept the change without a contributor agreement.
 1. Find someone to review the branch. Usually, this is a Launchpad developer you've already been talking to about the change. You can also go into the IRC channel `#launchpad-dev` on `irc.freenode.net` and see who's available to do review, or post to the development mailing list. See also [[Reviews|Code Review]].
 1. Propose your branch for merging, by going to https://code.launchpad.net/~my_username/launchpad/bugfix123 and click on '''Propose for merging into another branch'''. (The branch to merge into is `lp:~launchpad-pqm/launchpad/devel`.) Fill in the field for requested reviewer with the name of the reviewer you found, and follow the guidelines for writing the [[CoverLetters|merge cover letter]].
 1. Once the reviewer approves the change, merge it into Launchpad: {{{...working here...}}}		 #redirect PatchSubmission

Patches (last edited 2009-08-05 14:24:24 by kfogel)