Skip to content

Commit 01c0ef7

Browse files
authored
Merge pull request #312 from llucax/fix-309
Fix regression in `issue update`
2 parents a543760 + bda1716 commit 01c0ef7

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

git-hub

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1308,7 +1308,7 @@ class IssueCmd (CmdGroup):
13081308
(title, body) = split_titled_message(msg)
13091309
title_body['title'] = title
13101310
title_body['body'] = body
1311-
cls._do_update(url, args.label, args.assignee,
1311+
issue = cls._do_update(url, args.labels, args.assignee,
13121312
args.milestone, args.state, **title_body)
13131313
cls.print_issue_summary(issue)
13141314
@classmethod
@@ -1330,7 +1330,7 @@ class IssueCmd (CmdGroup):
13301330
params['title'] = title
13311331
if body:
13321332
params['body'] = body
1333-
issue = req.patch(url, **params)
1333+
return req.patch(url, **params)
13341334

13351335
class CommentCmd (IssueUtil):
13361336
cmd_help = "add a comment to an existing issue"

relnotes/issue-304.bug.md

Lines changed: 0 additions & 5 deletions
This file was deleted.
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
### Fix `issue update` regressions
2+
3+
A name error was raised when using the command `issue update`.
4+
5+
This is a regression introduced in v1.1.0.

0 commit comments

Comments
 (0)