Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an idiom #201

Merged
merged 1 commit into from
Mar 30, 2016
Merged

Add an idiom #201

merged 1 commit into from
Mar 30, 2016

Conversation

PeterDaveHello
Copy link
Contributor

No description provided.

@jserv
Copy link
Member

jserv commented Mar 30, 2016

The git commit message and content are inconsistent with each other.

@@ -92343,6 +92343,7 @@
狗尾續貂 0 ㄍㄡˇ ㄨㄟˇ ㄒㄩˋ ㄉㄧㄠ
狗屁 26 ㄍㄡˇ ㄆㄧˋ
狗屁不通 5 ㄍㄡˇ ㄆㄧˋ ㄅㄨˋ ㄊㄨㄥ
狗屁倒灶 10 ㄍㄡˇ ㄆㄧˋ ㄉㄠˇ ㄗㄠˋ
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought this is what I've done and record in the commit message, did I miss or misunderstand anything here?

Copy link
Member

Choose a reason for hiding this comment

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

The subject uses ㄆㄧˊ while the body takes ㄆㄧˋ, that is inconsistent. Please use git commit --amend to rework.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My bad ... sorry

@PeterDaveHello
Copy link
Contributor Author

@jserv I'm not sure how should I change the commit message, would you please give me your advice about that? Thanks!

@coveralls
Copy link

Coverage Status

Coverage remained the same at 89.498% when pulling 2ca50d7 on PeterDaveHello:add-an-idiom into 048bb26 on chewing:master.

@PeterDaveHello
Copy link
Contributor Author

Updated, sorry for my careless behavior.

@jserv jserv merged commit 48a2717 into chewing:master Mar 30, 2016
@PeterDaveHello PeterDaveHello deleted the add-an-idiom branch March 30, 2016 08:21
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