Merge lp:~angeloc/ubuntu-community-accomplishments/first-branch-merged into lp:~ubuntu-community-accomplishments-reviewers/ubuntu-community-accomplishments/0.1

Proposed by Angelo Compagnucci
Status: Merged
Merged at revision: 78
Proposed branch: lp:~angeloc/ubuntu-community-accomplishments/first-branch-merged
Merge into: lp:~ubuntu-community-accomplishments-reviewers/ubuntu-community-accomplishments/0.1
Diff against target: 128 lines (+109/-0)
4 files modified
accomplishments/ubuntu-community/en/canonical-contributor.accomplishment (+13/-0)
accomplishments/ubuntu-community/en/first-branch-merged.accomplishment (+24/-0)
scripts/ubuntu-community/canonical-contributor.py (+37/-0)
scripts/ubuntu-community/first-branch-merged.py (+35/-0)
To merge this branch: bzr merge lp:~angeloc/ubuntu-community-accomplishments/first-branch-merged
Reviewer Review Type Date Requested Status
Jono Bacon Needs Fixing
Review via email: mp+103659@code.launchpad.net

Description of the change

Check if you have merged any branch

To post a comment you must log in.
85. By Angelo Compagnucci

modified:
  accomplishments/ubuntu-community/en/first-branch-merged.accomplishment
   Fix typo

Revision history for this message
Jono Bacon (jonobacon) :
review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'accomplishments/ubuntu-community/en/canonical-contributor.accomplishment'
2--- accomplishments/ubuntu-community/en/canonical-contributor.accomplishment 1970-01-01 00:00:00 +0000
3+++ accomplishments/ubuntu-community/en/canonical-contributor.accomplishment 2012-04-26 10:53:21 +0000
4@@ -0,0 +1,13 @@
5+[accomplishment]
6+title=Signed the Canonical Contributors agreement
7+description=Signed the Canonical Contributors agreement and can have branches merged.
8+application=ubuntu-community
9+category=Development
10+icon=devel.png
11+depends=ubuntu-community/report-first-bug
12+needs-signing=true
13+needs-information=launchpad-email
14+summary = Canonical Contributor Agreement team tracks the set of people who have signed the Canonical Contributors agreement, and can have branches merged.
15+steps: Signing the single-page agreement is very easy and is typically done with a single email. For more information see attached links.
16+links: http://www.canonical.com/contributors
17+help = #ubuntu-devel on Freenode
18
19=== added file 'accomplishments/ubuntu-community/en/first-branch-merged.accomplishment'
20--- accomplishments/ubuntu-community/en/first-branch-merged.accomplishment 1970-01-01 00:00:00 +0000
21+++ accomplishments/ubuntu-community/en/first-branch-merged.accomplishment 2012-04-26 10:53:21 +0000
22@@ -0,0 +1,24 @@
23+[accomplishment]
24+title=First branch merged
25+description=Code you wrote is in Ubuntu, congratulations!
26+application=ubuntu-community
27+category=Development
28+icon=devel.png
29+depends=ubuntu-community/report-first-bug
30+needs-signing=true
31+needs-information=launchpad-email
32+summary = When you've come to a stage in your development where you're ready to merge your code into another branch you can make a public merge proposal.
33+steps: You submit your request to merge your branch into the main one (or whichever target branch interests you)
34+ Wait for one of the target branch's reviewers to review your code, they'll either:
35+ Accept your code, and it will be merged.
36+ Modify your code slightly, then merge it.
37+ Tell you what to do with a comment, marking your review as "Needs Fixing".
38+ If you're happy to, you do make the changes requested and push them into your branch. Then comment back on your proposal that you have done that. You should now await further action.
39+ Reject the changes. At this point, you can either try again discuss with the branch owner how to make your changes acceptable or continue work in your own branch.
40+tips: Join the IRC channel best suited for the branch you are working on and talk to developers, they will be glad to help you.
41+ Read bazaar documentation to get acquainted with it.
42+ Try to understand how code review works.
43+pitfalls: If your branch is not accepted, don't resign, try harder.
44+links: https://help.launchpad.net/Code/Review
45+ http://wiki.bazaar.canonical.com/Tutorials
46+help = #ubuntu-devel on Freenode
47
48=== added file 'scripts/ubuntu-community/canonical-contributor.py'
49--- scripts/ubuntu-community/canonical-contributor.py 1970-01-01 00:00:00 +0000
50+++ scripts/ubuntu-community/canonical-contributor.py 2012-04-26 10:53:21 +0000
51@@ -0,0 +1,37 @@
52+#!/usr/bin/python
53+import traceback, sys
54+
55+from accomplishments.daemon import dbusapi
56+
57+
58+try:
59+ import sys, os, pwd, subprocess
60+ from launchpadlib.launchpad import Launchpad
61+
62+ api = dbusapi.Accomplishments()
63+ f = api.get_extra_information("ubuntu-community", "launchpad-email")
64+ if bool(f[0]["launchpad-email"]) == False:
65+ sys.exit(4)
66+ else:
67+ email = f[0]["launchpad-email"]
68+
69+ # Check if user is member of contributor-agreement-canonical team,
70+ # using email to identify
71+ l = Launchpad.login_anonymously(
72+ 'ubuntu-community accomplishments', 'production')
73+ me = l.people.getByEmail(email=email)
74+ if me == None:
75+ sys.exit(1)
76+ else:
77+ team = l.people['contributor-agreement-canonical']
78+ members = team.getMembersByStatus(status='Approved')
79+ for member in members:
80+ if member.name == me.name:
81+ sys.exit(0)
82+ sys.exit(1)
83+
84+except SystemExit, e:
85+ sys.exit(e.code)
86+except:
87+ traceback.print_exc()
88+ sys.exit(2)
89
90=== added file 'scripts/ubuntu-community/first-branch-merged.py'
91--- scripts/ubuntu-community/first-branch-merged.py 1970-01-01 00:00:00 +0000
92+++ scripts/ubuntu-community/first-branch-merged.py 2012-04-26 10:53:21 +0000
93@@ -0,0 +1,35 @@
94+#!/usr/bin/python
95+import traceback, sys
96+
97+from accomplishments.daemon import dbusapi
98+
99+
100+try:
101+ import sys, os, pwd, subprocess
102+ from launchpadlib.launchpad import Launchpad
103+
104+ api = dbusapi.Accomplishments()
105+ f = api.get_extra_information("ubuntu-community", "launchpad-email")
106+ if bool(f[0]["launchpad-email"]) == False:
107+ sys.exit(4)
108+ else:
109+ email = f[0]["launchpad-email"]
110+
111+ # Check if user has at least one merge proposal with
112+ # 'Merged' status
113+ l = Launchpad.login_anonymously(
114+ 'ubuntu-community accomplishments', 'production')
115+ me = l.people.getByEmail(email=email)
116+ if me == None:
117+ sys.exit(1)
118+ else:
119+ branches = me.getMergeProposals(status='Merged')
120+ if len (branches) > 0:
121+ sys.exit(0)
122+ sys.exit(1)
123+
124+except SystemExit, e:
125+ sys.exit(e.code)
126+except:
127+ traceback.print_exc()
128+ sys.exit(2)

Subscribers

People subscribed via source and target branches

to all changes: