Skip to content

**削除禁止** レビューを再現したプルリク - #2

Draft
74616b756d69 wants to merge 1 commit into
mainfrom
example/needs-improvement
Draft

**削除禁止** レビューを再現したプルリク#2
74616b756d69 wants to merge 1 commit into
mainfrom
example/needs-improvement

Conversation

@74616b756d69

@74616b756d69 74616b756d69 commented Jul 24, 2026

Copy link
Copy Markdown
Owner

変更内容

自己紹介カードを新規追加しました。

  • 追加ファイル: exercises/participants/suzuki-ken.md
  • テンプレートに沿って、名前・自己紹介・好きな技術・ひとことを記載

変更の理由

GitHub ハンズオン演習の一環として、自己紹介カードを作成しました。
参加者同士の交流のきっかけになることを目的としています。

ビフォア・アフター

Before

exercises/participants/ に自分のファイルが存在しない状態

exercises/participants/
└── example-taro.md   ← サンプルのみ

After

自分の自己紹介カードが追加された状態

exercises/participants/
├── example-taro.md
└── suzuki-ken.md     ← 今回追加

動作確認方法

  1. exercises/participants/suzuki-ken.md をブラウザで開く
  2. Markdown が正しく表示されることを確認
    • 見出し(# ##)が適切にレンダリングされている
    • リスト(-)が箇条書きとして表示されている
  3. リンク切れがないことを確認

影響範囲

  • 影響あり: exercises/participants/ 配下に新規ファイル1つを追加
  • 影響なし: 既存ファイルの変更はなし。他の参加者のカードへの影響なし
  • 影響なし: ガイド本体(docs/README.md)への変更なし

チェックリスト

  • ファイル名が正しい(exercises/participants/suzuki-ken.md
  • テンプレートの形式に沿っている
  • Markdown の表示が崩れていない
  • 他の人のファイルを変更していない
  • 個人情報(電話番号・住所など)を含めていない
  • .DS_Store などの不要ファイルが含まれていない

これはサンプル PR です。PR の書き方の参考にしてください。
実務では「ビフォア・アフター」「動作確認方法」「影響範囲」を書くと、レビュアーが安心して確認できます。

@74616b756d69 74616b756d69 added the サンプル(マージしないで) 主催者が用意したサンプルPR。マージしないでください label Jul 24, 2026
@74616b756d69

Copy link
Copy Markdown
Owner Author

レビューさせていただきました!いくつか気になる点があります。

PR の書き方について

  • タイトルが「追加」だけだと、何を追加したのかわかりません
    → 例: add: 自己紹介カード(鈴木健)
  • 本文に「何を変更したか」「なぜ変更したか」「確認したこと」を書きましょう
    → PR テンプレートを参考にしてください

ファイルの内容について

  • 自己紹介をもう少し書いてみましょう(どんな仕事をしているか、何に興味があるか、など)
  • 「好きな技術」が「プログラミング」だけだと広すぎます
    → 例: Python(データ分析に使っています)React(個人開発で勉強中)
  • 「ひとこと」も、もう少し具体的だと嬉しいです

修正をお願いします!


参加者へ: これが「改善が必要な PR」の例です。タイトル・本文・内容のどこに問題があるか確認してみてください。

@74616b756d69 74616b756d69 changed the title 追加 add: 自己紹介カード(鈴木健) Jul 24, 2026
@74616b756d69

Copy link
Copy Markdown
Owner Author

レビューさせていただきました!

PR の書き方について

  • 変更内容・理由が明確に書かれていて良いです
  • ビフォア・アフターでフォルダ構成の変化がわかりやすい
  • 動作確認方法が具体的で、レビュアーが同じ手順で確認できます
  • 影響範囲が書かれているので、他に壊れるものがないと安心できます

ファイルの内容について

  • テンプレートに沿っていて OK です
  • Markdown の表示も問題ありません

LGTM!


参加者へ: これが実務に近い PR の書き方です。
特に「ビフォア・アフター」「動作確認方法」「影響範囲」は実務で重要です。
レビュアーが「この変更は安全か?」を判断するための材料になります。

@74616b756d69 74616b756d69 changed the title add: 自己紹介カード(鈴木健) **削除禁止** レビューを再現したプルリク Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

サンプル(マージしないで) 主催者が用意したサンプルPR。マージしないでください

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant