← Back to branch summary

~gmb/tarmac/handle-unicode-test-output-properly

« back to all changes in this revision

Viewing changes to HACKING

  • Committer: Maris Fogels
  • Date: 2010-11-04 16:34:30 UTC
  • mto: This revision was merged to the branch mainline in revision 383.
  • Revision ID: maris.fogels@canonical.com-20101104163430-1rr25nx39s2fkq01
Updated the hacking instructions based on reviewer feedback - draft number 2

Show diffs side-by-side

added added

removed removed

Lines of Context:
HACKING
16
16
 
17
17
2. Push your changes: ``bzr push lp:~myuser/tarmac/my-descriptive-patch``
18
18
 
19
 
3. Use ``bzr send`` or the Launchpad web interface to propose your branch for
20
 
   merging into the Tarmac trunk.
21
 
 
22
 
4. Wait for a Tarmac committer to review the code and reply to your proposal.
23
 
 
24
 
5. If you have been waiting a long time for someone to review your code, join
25
 
   the #tarmac IRC channel on FreeNode and ask rockstar and dobey personally
26
 
   to look at your branch.
27
 
 
28
 
6. When your code is ready to land, set the merge proposal's status to
29
 
   'Approved'. The Tarmac project's own instance will land the changes for
30
 
   you.
 
19
3. Follow the Launchpad help instructions to create a new merge proposal
 
20
   containing your changes: https://help.launchpad.net/Code/Review
 
21
 
 
22
4. The Tarmac core developers will review your code and provide feedback.
 
23
   When your code is approved they will commit it for you.
31
24
 
32
25
 
33
26
=============