/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/plugins/propose/github.py

  • Committer: Jelmer Vernooij
  • Date: 2019-09-21 17:08:09 UTC
  • mfrom: (7389 work)
  • mto: This revision was merged to the branch mainline in revision 7390.
  • Revision ID: jelmer@jelmer.uk-20190921170809-ejewbeue585deajo
merge trunk.

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/plugins/propose/github.py
18
18
 
19
19
from __future__ import absolute_import
20
20
 
 
21
import json
21
22
import os
22
23
 
23
24
from .propose import (
28
29
    MergeProposalExists,
29
30
    NoSuchProject,
30
31
    PrerequisiteBranchUnsupported,
 
32
    ReopenFailed,
31
33
    UnsupportedHoster,
32
34
    )
33
35
 
72
74
    return description.splitlines()[0]
73
75
 
74
76
 
 
77
class ValidationFailed(errors.BzrError):
 
78
 
 
79
    _fmt = "GitHub validation failed: %(error)s"
 
80
 
 
81
    def __init__(self, error):
 
82
        errors.BzrError.__init__(self)
 
83
        self.error = error
 
84
 
 
85
 
75
86
class NotGitHubUrl(errors.BzrError):
76
87
 
77
88
    _fmt = "Not a GitHub URL: %(url)s"
107
118
 
108
119
class GitHubMergeProposal(MergeProposal):
109
120
 
110
 
    def __init__(self, pr):
 
121
    def __init__(self, gh, pr):
 
122
        self._gh = gh
111
123
        self._pr = pr
112
124
 
 
125
    def __repr__(self):
 
126
        return "<%s at %r>" % (type(self).__name__, self.url)
 
127
 
113
128
    @property
114
129
    def url(self):
115
130
        return self._pr['html_url']
116
131
 
117
132
    def _branch_from_part(self, part):
 
133
        if part['repo'] is None:
 
134
            return None
118
135
        return github_url_to_bzr_url(part['repo']['html_url'], part['ref'])
119
136
 
120
137
    def get_source_branch_url(self):
129
146
    def get_commit_message(self):
130
147
        return None
131
148
 
 
149
    def set_commit_message(self, message):
 
150
        self._patch({'title': message})
 
151
 
 
152
    def _patch(self, data):
 
153
        response = self._gh._api_request(
 
154
            'PATCH', self._pr['url'], body=json.dumps(data).encode('utf-8'))
 
155
        if response.status == 422:
 
156
            raise ValidationFailed(json.loads(response.text))
 
157
        if response.status != 200:
 
158
            raise InvalidHttpResponse(self._pr['url'], response.text)
 
159
        self._pr = json.loads(response.text)
 
160
 
132
161
    def set_description(self, description):
133
 
        self._pr.edit(body=description, title=determine_title(description))
 
162
        self._patch({
 
163
            'body': description,
 
164
            'title': determine_title(description),
 
165
            })
134
166
 
135
167
    def is_merged(self):
136
 
        return self._pr['merged']
 
168
        return bool(self._pr.get('merged_at'))
 
169
 
 
170
    def is_closed(self):
 
171
        return self._pr['state'] == 'closed' and not bool(self._pr.get('merged_at'))
 
172
 
 
173
    def reopen(self):
 
174
        try:
 
175
            self._patch({'state': 'open'})
 
176
        except ValidationFailed as e:
 
177
            raise ReopenFailed(e.error['errors'][0]['message'])
137
178
 
138
179
    def close(self):
139
 
        self._pr.edit(state='closed')
 
180
        self._patch({'state': 'closed'})
 
181
 
 
182
    def can_be_merged(self):
 
183
        return self._pr['mergeable']
140
184
 
141
185
    def merge(self, commit_message=None):
142
186
        # https://developer.github.com/v3/pulls/#merge-a-pull-request-merge-button
177
221
    def __repr__(self):
178
222
        return "GitHub()"
179
223
 
180
 
    def _api_request(self, method, path):
 
224
    def _api_request(self, method, path, body=None):
181
225
        headers = {
182
226
            'Accept': 'application/vnd.github.v3+json'}
183
227
        if self._token:
184
228
            headers['Authorization'] = 'token %s' % self._token
185
229
        response = self.transport.request(
186
230
            method, urlutils.join(self.transport.base, path),
187
 
            headers=headers)
 
231
            headers=headers, body=body, retries=3)
188
232
        if response.status == 401:
189
233
            raise GitHubLoginRequired(self)
190
234
        return response
195
239
        if response.status == 404:
196
240
            raise NoSuchProject(path)
197
241
        if response.status == 200:
198
 
            return response.json
199
 
        raise InvalidHttpResponse(path, response.text)
 
242
            return json.loads(response.text)
 
243
        raise InvalidHttpResponse(path, response.text)
 
244
 
 
245
    def _get_repo_pulls(self, path, head=None, state=None):
 
246
        path = 'repos/' + path + '/pulls?'
 
247
        params = {}
 
248
        if head is not None:
 
249
            params['head'] = head
 
250
        if state is not None:
 
251
            params['state'] = state
 
252
        path += ';'.join(['%s=%s' % (k, urlutils.quote(v))
 
253
                         for k, v in params.items()])
 
254
        response = self._api_request('GET', path)
 
255
        if response.status == 404:
 
256
            raise NoSuchProject(path)
 
257
        if response.status == 200:
 
258
            return json.loads(response.text)
 
259
        raise InvalidHttpResponse(path, response.text)
 
260
 
 
261
    def _create_pull(self, path, title, head, base, body=None):
 
262
        path = 'repos/' + path + '/pulls'
 
263
        data = {
 
264
            'title': title,
 
265
            'head': head,
 
266
            'base': base,
 
267
        }
 
268
        if body:
 
269
            data['body'] = body
 
270
 
 
271
        response = self._api_request(
 
272
            'POST', path, body=json.dumps(data).encode('utf-8'))
 
273
        if response.status != 201:
 
274
            raise InvalidHttpResponse(path, response.text)
 
275
        return json.loads(response.text)
 
276
 
 
277
    def _get_user_by_email(self, email):
 
278
        path = 'search/users?q=%s+in:email' % email
 
279
        response = self._api_request('GET', path)
 
280
        if response.status != 200:
 
281
            raise InvalidHttpResponse(path, response.text)
 
282
        ret = json.loads(response.text)
 
283
        if ret['total_count'] == 0:
 
284
            raise KeyError('no user with email %s' % email)
 
285
        elif ret['total_count'] > 1:
 
286
            raise ValueError('more than one result for email %s' % email)
 
287
        return ret['items'][0]
200
288
 
201
289
    def _get_user(self, username=None):
202
290
        if username:
206
294
        response = self._api_request('GET', path)
207
295
        if response.status != 200:
208
296
            raise InvalidHttpResponse(path, response.text)
209
 
        return response.json
 
297
        return json.loads(response.text)
210
298
 
211
299
    def _get_organization(self, name):
212
300
        path = 'orgs/:%s' % name
213
301
        response = self._api_request('GET', path)
214
302
        if response.status != 200:
215
303
            raise InvalidHttpResponse(path, response.text)
216
 
        return response.json
 
304
        return json.loads(response.text)
217
305
 
218
306
    def _search_issues(self, query):
219
307
        path = 'search/issues'
221
309
            'GET', path + '?q=' + urlutils.quote(query))
222
310
        if response.status != 200:
223
311
            raise InvalidHttpResponse(path, response.text)
224
 
        return response.json
 
312
        return json.loads(response.text)
225
313
 
226
314
    def _create_fork(self, repo, owner=None):
227
315
        (orig_owner, orig_repo) = repo.split('/')
231
319
        response = self._api_request('POST', path)
232
320
        if response != 202:
233
321
            raise InvalidHttpResponse(path, response.text)
234
 
        return response.json
 
322
        return json.loads(response.text)
235
323
 
236
324
    @property
237
325
    def base_url(self):
303
391
            parse_github_branch_url(source_branch))
304
392
        (target_owner, target_repo_name, target_branch_name) = (
305
393
            parse_github_branch_url(target_branch))
306
 
        target_repo = self._get_repo(
307
 
            "%s/%s" % (target_owner, target_repo_name))
 
394
        target_repo_path = "%s/%s" % (target_owner, target_repo_name)
 
395
        target_repo = self._get_repo(target_repo_path)
308
396
        state = {
309
397
            'open': 'open',
310
398
            'merged': 'closed',
311
399
            'closed': 'closed',
312
400
            'all': 'all'}
313
 
        for pull in target_repo.get_pulls(
314
 
                head=target_branch_name,
315
 
                state=state[status]):
316
 
            if (status == 'closed' and pull.merged or
317
 
                    status == 'merged' and not pull.merged):
318
 
                continue
319
 
            if pull.head.ref != source_branch_name:
320
 
                continue
321
 
            if pull.head.repo is None:
 
401
        pulls = self._get_repo_pulls(
 
402
            target_repo_path,
 
403
            head=target_branch_name,
 
404
            state=state[status])
 
405
        for pull in pulls:
 
406
            if (status == 'closed' and pull['merged'] or
 
407
                    status == 'merged' and not pull['merged']):
 
408
                continue
 
409
            if pull['head']['ref'] != source_branch_name:
 
410
                continue
 
411
            if pull['head']['repo'] is None:
322
412
                # Repo has gone the way of the dodo
323
413
                continue
324
 
            if (pull.head.repo.owner.login != source_owner or
325
 
                    pull.head.repo.name != source_repo_name):
 
414
            if (pull['head']['repo']['owner']['login'] != source_owner or
 
415
                    pull['head']['repo']['name'] != source_repo_name):
326
416
                continue
327
 
            yield GitHubMergeProposal(pull)
 
417
            yield GitHubMergeProposal(self, pull)
328
418
 
329
419
    def hosts(self, branch):
330
420
        try:
361
451
            query.append('is:merged')
362
452
        query.append('author:%s' % self._current_user['login'])
363
453
        for issue in self._search_issues(query=' '.join(query))['items']:
364
 
            yield GitHubMergeProposal(
365
 
                self.transport.request('GET', issue['pull_request']['url']).json)
 
454
            url = issue['pull_request']['url']
 
455
            response = self._api_request('GET', url)
 
456
            if response.status != 200:
 
457
                raise InvalidHttpResponse(url, response.text)
 
458
            yield GitHubMergeProposal(self, json.loads(response.text))
366
459
 
367
460
    def get_proposal_by_url(self, url):
368
461
        raise UnsupportedHoster(url)
406
499
        # TODO(jelmer): Probe for right repo name
407
500
        if self.target_repo_name.endswith('.git'):
408
501
            self.target_repo_name = self.target_repo_name[:-4]
409
 
        target_repo = self.gh._get_repo("%s/%s" % (self.target_owner, self.target_repo_name))
410
502
        # TODO(jelmer): Allow setting title explicitly?
411
503
        title = determine_title(description)
412
 
        # TOOD(jelmer): Set maintainers_can_modify?
 
504
        # TODO(jelmer): Set maintainers_can_modify?
413
505
        try:
414
 
            pull_request = target_repo.create_pull(
 
506
            pull_request = self.gh._create_pull(
 
507
                "%s/%s" % (self.target_owner, self.target_repo_name),
415
508
                title=title, body=description,
416
509
                head="%s:%s" % (self.source_owner, self.source_branch_name),
417
510
                base=self.target_branch_name)
418
 
        except github.GithubException as e:
419
 
            if e.status == 422:
420
 
                raise MergeProposalExists(self.source_branch.user_url)
421
 
            raise
 
511
        except ValidationFailed:
 
512
            raise MergeProposalExists(self.source_branch.user_url)
422
513
        if reviewers:
423
514
            for reviewer in reviewers:
424
 
                pull_request.assignees.append(
425
 
                    self.gh._get_user(reviewer)['login'])
 
515
                if '@' in reviewer:
 
516
                    user = self.gh._get_user_by_email(reviewer)
 
517
                else:
 
518
                    user = self.gh._get_user(reviewer)
 
519
                pull_request.assignees.append(user['login'])
426
520
        if labels:
427
521
            for label in labels:
428
522
                pull_request.issue.labels.append(label)
429
 
        return GitHubMergeProposal(pull_request)
 
523
        return GitHubMergeProposal(self.gh, pull_request)