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

fixed bug #14

Merged
merged 1 commit into from
Apr 11, 2021
Merged

fixed bug #14

merged 1 commit into from
Apr 11, 2021

Conversation

HonokaNo
Copy link
Contributor

@HonokaNo HonokaNo commented Apr 9, 2021

issueより

@uchan-nos
Copy link
Owner

HonokaNo さん
プルリクありがとうございます。

コミットコメントが「fixed bug」だけでは、何を直したのか分かりません。
コミットコメントを次の観点で修正して、force push(既存のコミットを上書き)してもらえませんか?

なお、英語が苦手であれば日本語のコミットコメントでも全然かまいません。

  • 特に英語の場合は三人称単数現在形で書く。今回の場合なら「fixes」とか。
    • これは、「このコミットは~をする」というように、このコミットを採用すると起きることを説明するためです。
  • コミットタイトルは、コミットの概要を簡単に書く。例えば「ExitBootServices の前に Print を使う」
  • コミット本文は、必要があればさらにコミットを説明する。今回は簡単な変更なので不要かなと。
    • あるいは、 Print(L"All done\n"); はそのまま残している理由について書いてもいいかもしれませんね。「ここは実行されないはずだから修正しない。」とか。

@uchan-nos
Copy link
Owner

このコミットコメントなんかは、とてもシンプルですが概要を表していて良いですね。
#16

@HonokaNo
Copy link
Contributor Author

コミットコメントを変更しました。
P.S.
Print(L"All done\n");はとりあえず残しておきました。
あくまでエラーメッセージが表示されないのは問題であろう、というための修正です。

@uchan-nos uchan-nos merged commit a68d282 into uchan-nos:master Apr 11, 2021
@uchan-nos
Copy link
Owner

プルリクありがとうございました!

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