Skip to content
New issue

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

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

Already on GitHub? # to your account

おめシス人気投票リンクの遷移先を変更 #370

Merged
merged 1 commit into from
Feb 13, 2020
Merged

おめシス人気投票リンクの遷移先を変更 #370

merged 1 commit into from
Feb 13, 2020

Conversation

ryamaguchi0220
Copy link
Contributor

@ryamaguchi0220 ryamaguchi0220 commented Feb 11, 2020

変更内容: Summary

おめシス人気投票リンクの遷移先を #368 で追加された人気投票セクションに変更します。

関連 : #369

確認事項: Check point

  • PR を作成する前に、 https://github.com/omegasisters/homepage の最新の master を取り込み済みである。
    • Conflict や他の方の変更で自分の変更が動かなくなる可能性を防ぎます。
    • 最新の master を取り込む方法
      • upstream に fork 元リポジトリを追加
        • git remote add upstream git@github.com:omegasisters/homepage.git
      • 現在のブランチに upstreammaster を取り込む
        • $ git pull --rebase upstream master
    • おまけ
      • rebase 後に再度 push する場合、 --force-with-lease オプションをつける
        • git push --force-with-lease origin <ブランチ名>
  • 動作確認済みである。
    • 何らかの理由で本番に取り込まれるまで確認できない場合はその旨を補足に記載する。
  • prettier によるコード整形を行った、もしくは画面に関係ない変更である。
    • 可能な方のみで良いと思いますが、意図せず他の方がフォーマットするとコード差分が増えすぎるので自分の分は自分でやるのがよろしいかと思います。
  • スマホ(狭い画角)でも表示を確認した、もしくは画面に関係ない変更である。
  • 他の方の変更を意図せず削除・変更していないか、差分をもう一度確認した。
  • 破壊的な変更を行った場合、影響範囲をもう一度確認した。もしくは破壊的な変更を行っていない。
  • Pull Request に関連した issue の URL を貼り付けた

補足: Other Information

@omegasisters omegasisters merged commit 68f0fa8 into omegasisters:master Feb 13, 2020
@omegasisters
Copy link
Owner

👍

@ryamaguchi0220 ryamaguchi0220 deleted the change-popularity-poll-link-href branch February 13, 2020 05:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants