Skip to content

Fix review message parsing from editor failed#133

Merged
nhance merged 1 commit intoreenhanced:masterfrom
francois:master
Sep 10, 2015
Merged

Fix review message parsing from editor failed#133
nhance merged 1 commit intoreenhanced:masterfrom
francois:master

Conversation

@francois
Copy link
Copy Markdown
Contributor

@francois francois commented Sep 4, 2015

In the editor, when I wrote out

a

b

the pull request's title only "b", and there was no body:

Review your PR:
--------
Title:
b

Body:

--------
Submit pull request? (Y)

With this change, I get the expected title and body:

Review your PR:
--------
Title:
a

Body:
b
--------
Submit pull request? (Y)

In the editor, when I wrote out

```
a

b
```

the pull request's title only "b", and there was no body:

```
Review your PR:
--------
Title:
b

Body:

--------
Submit pull request? (Y)
```

With this change, I get the expected title and body:

```
Review your PR:
--------
Title:
a

Body:
b
--------
Submit pull request? (Y)
```
@nhance
Copy link
Copy Markdown
Member

nhance commented Sep 4, 2015

lgtm

(We'll likely merge in next week, this is a holiday weekend for us)

nhance added a commit that referenced this pull request Sep 10, 2015
Fix review message parsing from editor failed
@nhance nhance merged commit 43a5065 into reenhanced:master Sep 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants