Skip to content

Fix commit message overflow#623

Merged
miguelsolorio merged 2 commits intomicrosoft:masterfrom
JeffreyCA:commit-msg-overflow
Oct 30, 2018
Merged

Fix commit message overflow#623
miguelsolorio merged 2 commits intomicrosoft:masterfrom
JeffreyCA:commit-msg-overflow

Conversation

@JeffreyCA
Copy link
Copy Markdown
Member

@JeffreyCA JeffreyCA commented Oct 26, 2018

Resolves #609 and #618

screen shot 2018-10-26 at 12 56 11 am

Copy link
Copy Markdown

@miguelsolorio miguelsolorio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@miguelsolorio miguelsolorio merged commit a0866e3 into microsoft:master Oct 30, 2018
@JeffreyCA
Copy link
Copy Markdown
Member Author

Not sure what happened, 2 of my PRs (this and #645) are no longer in the current master branch. None of these changes can be seen in master.

@miguelsolorio
Copy link
Copy Markdown

@JeffreyCA we're looking into it, it's quite strange that this was clearly merged above but there is no trace of it in master.

@rebornix
Copy link
Copy Markdown
Member

rebornix commented Nov 5, 2018

@JeffreyCA I cherry picked your commits to master and added disable-force-push rule to master, let's see if this happens again or not.

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.

3 participants