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

Update index.html #5

Merged
merged 1 commit into from Mar 16, 2016

Conversation

Projects
None yet
2 participants
@plalayout
Contributor

plalayout commented Mar 16, 2016

小文字でl(L)を入力した場合に反映されない箇所を修正

Update index.html
小文字でl(L)を入力した場合に反映されない箇所を修正
@gam0022

This comment has been minimized.

Show comment
Hide comment
@gam0022

gam0022 Mar 16, 2016

Owner

@plalayout さん
ついにPRの作り方をマスターされましたね!
内容は完璧です。

少しだけ惜しい点があるので、次回からは気をつけて欲しいです。今回はOKです。

  • 「Update index.html」というPR名では内容が分からないので、「小文字でl(L)を入力した場合に反映されない箇所を修正」など、具体的なPR名にしてください。
  • ブランチ名は、feature/permalink ではなく、fix/lowercase-l-ignored など、実態に合ったブランチ名にしましょう。
Owner

gam0022 commented Mar 16, 2016

@plalayout さん
ついにPRの作り方をマスターされましたね!
内容は完璧です。

少しだけ惜しい点があるので、次回からは気をつけて欲しいです。今回はOKです。

  • 「Update index.html」というPR名では内容が分からないので、「小文字でl(L)を入力した場合に反映されない箇所を修正」など、具体的なPR名にしてください。
  • ブランチ名は、feature/permalink ではなく、fix/lowercase-l-ignored など、実態に合ったブランチ名にしましょう。

gam0022 added a commit that referenced this pull request Mar 16, 2016

@gam0022 gam0022 merged commit d509be7 into gam0022:master Mar 16, 2016

@plalayout

This comment has been minimized.

Show comment
Hide comment
@plalayout

plalayout Mar 16, 2016

Contributor

コメントありがとうございます。まだまだ至らないようですが、よろしくお願いします。

Contributor

plalayout commented Mar 16, 2016

コメントありがとうございます。まだまだ至らないようですが、よろしくお願いします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment