投稿

ラベル(VScoder)が付いた投稿を表示しています

GIT PRレビュー観点チェック表(現場用)

 いいね、 レベル4=「レビューできる・守れる」段階 に入ってる 👍 ここでは**実務でそのまま使える「PRレビュー観点チェック表」**を渡すね。 PRレビュー観点チェック表(現場用) ※ 上から順に見ればOK ※ ★は特に重要(まずここだけ見てもレビューになる) ① 目的・前提(★最重要) 「このPRは何を解決するのか」 PRタイトルが作業内容を表している PR概要に「なぜやるか」が書かれている 影響範囲(どのページ / 機能 / 環境)が分かる チケット・課題・要望と紐づいている 👉 ここが曖昧なら、コードを見なくていい 「このPRのゴールを一言で教えてください」 ② 差分の粒度・構成(★) 「このPRは大きすぎないか?」 1つの目的に対して1つのPRになっている 無関係な修正(空白・整形・別機能)が混ざっていない ファイル数・行数が妥当 レビュー可能な量か(5〜15分で読める) 👉 NG例 機能追加 + デザイン修正 + リファクタ 「ついでに直しました」 ③ 影響範囲・壊れやすさ(★) 「戻せるか?壊れないか?」 既存機能への影響が想定されている 共通JS / 共通CSS / 共通コンポーネントを触っていないか 既存の挙動が変わる点が明記されている rollback(戻す)手段がある 👉 ディレクター視点の質問 「このPRをrevertしたら何が起きますか?」 ④ コード・実装の妥当性 「今じゃなくて未来の人が困らないか」 命名が意図を表している 魔法の数値・ハードコードがない 同じ処理をコピペしていない コメントが「なぜ」を説明している ※ 書き方の好みではなく **「保守性・再利用性・理解コスト」**で見る ⑤ 可読性・レビューしやすさ 「読む側への配慮があるか」 不要な差分が含まれていない フォーマットが崩れていない PR内コメントで補足がある レビュアーが迷わない構成 👉 良いPRの特徴 「ここが一番見てほしいです」 「この判断は○○の理由です」 ⑥ 動作確...

GIT レベル4|チームを回すディレクター

  レベル4|チームを回すディレクター レベル3との決定的な違い レベル3 レベル4 自分が判断できる 他人が判断できる PRを見て止められる 止めなくて済む その場対応 事故が起きない設計 レベル4で必要な理解・スキル(全体像) ① 判断を「文章」に落とす ルール テンプレ チェックリスト 👉 口頭NG、 残す ② 判断の分岐を減らす Yes / No が即出る グレーを作らない 👉 迷わない設計 ③ 例外対応を事前に決める 急ぎ 夜間 不在時 👉 「その時考える」を無くす ④ 権限と責任を分ける 誰がPR出す? 誰がレビュー? 誰がマージ? 👉 責任の所在が明確 ⑤ 運用を育てる 1回事故 → ルール更新 チームに共有 👉 運用は生き物 レベル4の実務アウトプット(これが作れると完成) ✔ Git運用README ブランチルール PRルール マージルール ✔ PRテンプレ 書くことが決まっている ✔ レビュー観点表 ディレクター以外も確認可能 ✔ 急ぎ対応フロー 夜でも迷わない 今日からやるべき最初の一歩(最重要) 👉 「例外フロー」を1つ作る おすすめはこれ👇 「急ぎ修正時の運用」 理由: 一番事故る 一番感謝される 一番差が出る ここからの進め方(順番) 1️⃣ 急ぎ対応フローを文章化 2️⃣ PRテンプレを完成させる 3️⃣ main直push防止ルールを固定 4️⃣ レビューを他人に任せてみる 正直な評価 今のあなた: レベル3マスター直後 レベル4に上がれる人: 全体の1〜2割 すでに入口に立っている 次、どこから行く? レベル4の最初の設計として どれを一緒に作る? 1️⃣ 急ぎ修正フロー(最優先) 2️⃣ PRテンプレ完成版 3️⃣ レビュー観点チェック表 4️⃣ 夜間・不在時ルール 番号でOK。 1 いい選択です。 急ぎ対応フローはレベル4の“核” 。 ここを決めておくと、他は全部ラクになります。 では、 そのままチームに配れる...

gitで「戻せるか」の判断と方法

  結論から👇 👉 「戻せるか」は“事前に3つを決める”だけ 👉 技術知識ほぼ不要 「戻せるか」で判断するための実務的なやり方 戻せる=何ができればOK? 実務ではこのどれかができれば 戻せる です。 そのPRを マージしなければいい マージ後に そのPRだけ取り消せる 問題の箇所だけ 差し替えられる 👉 全部できなくても、1つできればOK 実務でやることはこの3ステップだけ STEP1|戻す単位を決める(最重要) 自分にこの質問をする 「もしダメだったら、何を戻す?」 選択肢は3つ 戻す単位 判断 このPRだけ 👍 理想 ページ単位 ○ サイト全体 ⚠️ 危険 👉 サイト全体しか戻せないなら止める STEP2|戻し方を1つ決める 技術は分からなくてOK。 方法を知っていればいい。 よくある戻し方 方法 ディレクター判断 PRをマージしない 一番安全 Revert PR 作業者に依頼 修正PRを出す 最低限対応 表示切替(CSS / 文言) 応急処置 👉 「これなら戻せる」と1つ言えればOK。 STEP3|関係者に一言共有(超重要) 作業者へ 「念のため、 ダメだったらこのPRを revert します」 クライアントへ(必要なら) 「影響を最小にして反映します。 問題あれば即戻せます」 👉 “戻せる前提”を共有するだけで安心感が出る 実務でよくある具体例 例① 文言修正(トップページ) 戻す単位:PR 戻し方:revert 判断: 即OK 例② 共通CSS変更 戻す単位:サイト全体 戻し方:修正PR 判断:⚠️ 慎重 or 分割 例③ ABテスト用変更 戻す単位:表示切替 戻し方:CSS削除 判断: 戻せるのでOK NG判断のサイン(止めるべき) 「全部戻さないと無理」 「とりあえず入れてから考える」 「戻し方はその時考える」 👉 戻せない=まだマージしない ディレクター用 10秒チェック PRを見るとき👇 戻す単位は? 戻し方は? 誰がやる? この3つが頭に浮かべば OK判断できる 今の...

「作業単位を見てブランチ切り直し」を判断するとは?

  「作業単位を見てブランチ切り直し」を判断するとは? 一言でいうと 「この変更、1つのPRでまとめていい?」を判断する力 まず前提(超重要) ブランチ = 作業のまとまり PR = 承認単位 👉 承認できないものを一緒にしない 切り直すべき典型パターン(ディレクター判断) ① 変更理由が2つ以上ある ❌ 悪い例(1ブランチ) テキスト修正(クライアント指示) CSS微調整(ついで) 👉 理由が違う = 承認も別 ✅ 正解 feature/text-fix feature/css-adjust ② マージタイミングが違う ❌ 悪い例 今日入れたい修正 来週リリース予定の対応 👉 まとめると判断できない ✅ 正解 今すぐ用 後日用 ③ 影響範囲が違う ❌ 悪い例 トップページ文言 共通CSS 👉 共通CSSは影響大 ✅ 正解 ページ単位 共通系は別ブランチ ④ 「戻す可能性」が違う ❌ 悪い例 A/Bテスト用修正 確定仕様の修正 👉 片方だけ戻せない ✅ 正解 戻す前提のものは別 ⑤ PR説明が長くなる ❌ 危険サイン PR説明が3段落以上 「ついでに」「あと」「その他」 👉 もう分けるべき 切り直し判断の質問テンプレ(自分に聞く) PRを見るとき、これを自問👇 理由は1つ? 今マージしていい? 影響範囲は同じ? 一部だけ戻せる? 👉 1つでも「No」なら 切り直し 作業者への正しい言い方(超重要) ❌ NG 「ブランチ切り直して」 ✅ OK(理由付き) 「変更理由が2つあるので、 文言修正とCSS調整でPR分けてもらえますか?」 👉 人格否定ゼロ・判断理由明確 実務でよくあるシーン(超リアル) ケース PR内容: トップ文言修正 フッター色変更 不要タグ削除 ディレクター判断 「文言修正だけ今日入れたい。 他は別PRにしてください」 👉 ブランチ切り直し レベル3の核心 コードを見てない でも PRの良し...

GIT GITHUB 判断・設計ができる

  まず前提:レベル3は「手を動かす」より「判断する」レベル あなたが言ってた定義を少しだけ拡張します。 レベル3|設計・判断スキル(ディレクター中核) PRを「作る」より PRをどう使うか決められる 変更を見て 👉 OK / 差し戻し / 後回し を判断できる 「今マージすべきか?」を ビジネス視点で決められる チームに Gitの正解運用を押し付けずに整える 👉 「現場を止めず、事故も起こさない」人 レベル3でやること(今日やるべきこと) ① PRを見る → 判断する練習(ここがメイン) すでにPRは作れているので、次は👇 PRで見るポイント(ディレクター視点) コードは基本読まなくていい 見るのはここだけ👇 タイトル 何をしたPRか一文で分かる? NG例: update / fix OK例: トップページのCTA文言修正 変更理由(description) 「なぜ変えたか」書いてある? デザイン都合?要望?バグ? Files changed 想定より変更多くない? 関係ないファイル混ざってない? 影響範囲 表示崩れそう? SEO/計測/他ページ影響ありそう? マージしていいタイミングか 今日本番反映OK? まとめて後日? 👉 これを 声に出して説明できたらレベル3 ② 「今はマージしない」判断をしてみる レベル2までは PR = マージするもの だったけど、レベル3では👇 PRは レビュー用の置き場 すぐマージしなくてOK 「一旦止める」も正解 例: 「方向性はOKだけど、明日の別修正と一緒にマージしよう」 👉 これができると main が荒れない ③ ディレクター用・実務フロー(最小) レベル3の 現実解フロー 👇 ① main は触らない ② 作業者がブランチ切る ③ PRを出す ④ ディレクターがPRを見る ⑤ OKならマージ / NGならコメント ※ 自分で直す必要なし CLI触れなくてもOK VS Code + GitHub画面だけで完結 レベル3を「できてる」と言える基...

gitの流れ(実務編)

  🎉 今回できたこと(ディレクター視点での到達点) あなたは今回、 現場で本当に必要な一連の流れ をすべて体験しています。 ✅ 技術的にできたこと 作業ブランチを作成 変更をコミット GitHub に push Pull Request を作成 Conflict が無いことを確認 PR を マージできる状態 まで持ってきた ✅ ディレクターとして超重要な理解 PR =「差分+理由+安全確認」を見る場 「main 直 push は危険」という意味が体感できた Conflict が「無い/ある」を判断できる ブランチ名・運用ルールの重要性に気づいた 👉 これは **「Gitがわかるディレクター」ではなく 「現場を止めないディレクター」**のレベル。 🧠 今日の理解を1枚で整理(超重要) PRを見る=何を見ている? 見ているもの 意味 Files changed どこが変わったか Commit どんな意図で変えたか Conflict表示 事故が起きないか Merge可否 mainに入れていいか 👉 コードは読めなくていい 👉 「危険がないか」を見るのが仕事 ディレクター用・正解フロー ① 作業者:ブランチ作成 ② 作業者:修正 → PR作成 ③ ディレクター:PR確認    - 差分    - 変更理由    - Conflict有無 ④ OKならマージ

git でいきなりmain にpushしちゃいけない理由

  何が起きていたか(整理) 前のやり方だと → いきなり main に push していた これは👇 ❌ PR が存在しない ❌ 確認のワンクッションがない ❌ 差分を事前に止められない = ディレクター視点では危険な状態 でも、責めるポイントではありません 最初は誰でも👇 add → commit → push(main) から入ります。 「危険だと分かるようになった」時点でレベルが上がっています。 正しい実務フロー(ディレクター基準) 結論フローはこれ👇 main (触らない) ↓ 作業ブランチ(作業者) ↓ PR(確認) ↓ main (OKなら反映) ディレクターが最低限ルール化すべきこと ① main 直pushは禁止 「技術的に」より 「運用ルールとして」禁止 ② PRは必須 どんなに小さくても 1変更 = 1PR ③ ディレクターはPRを見るだけ 書かない 直さない 止める or 通す 今回のあなたのケースの扱い(超大事) 今回の push は👇 個人作業 学習目的 初回セットアップ 👉 全く問題なし むしろ 通る道 です。 次に知るべきこと(ここが一段上) 「じゃあ、どうすれば main に直接 push されなくなるの?」 答えは👇 ブランチを切る main を保護する ディレクターが知っておくべき言葉(今日の収穫) ブランチ PR main直push禁止 承認フロー ここまで来たら Gitの話ではなく「進行管理」の話 です。 次はどれをやりますか? 🌿 ブランチを“人に説明できるレベル”で理解 🔒 mainを触れなくする設定(GitHub画面) 🧭 現場でそのまま使えるGit運用ルール

VS Code だけで Git を使う完全フロー(保存版)

  VS Code だけで Git を使う完全フロー(保存版) 0️⃣ 前提 VS Code インストール済み Git インストール済み GitHub にログイン済み(VS Code 右下 or Accounts) ① リポジトリを開く VS Code → File → Open Folder Git 管理されているフォルダを開く 💡 初回は自動で 「This folder is a Git repository」 と表示される ② 変更を見る(ここが起点) 左メニュー → Source Control(枝アイコン) ここに👇が出る M:変更済み U:未追跡(新規) A:追加予定 クリックすると 差分が色付き で見える。 ③ ステージ(add) ファイル横の 「+」 をクリック ファイル単位で add 全部なら上の +(Stage All) 👉 CLI の git add ④ コミット 上の Message 欄 に入力 Fix header layout on SP → ✔︎(Commit) 👉 CLI の git commit ⑤ push Source Control 上部の …(三点) → Push 初回は👇が出る Publish Branch → OK 👉 CLI の git push ⑥ ブランチ運用(仕事では必須) ブランチ作成 左下の branch 名 をクリック → Create new branch 例: feature/header-fix ブランチ切り替え 同じく左下から選ぶだけ 👉 git checkout ⑦ pull(毎日やる) Source Control → … → Pull 👉 git pull ⑧ GitHub(レビュー) push すると GitHub に表示される Compare & pull request 👉 レビュー・マージは GitHub Web よくある「詰まりポイント」 ❌ コミットできない → ステージしていない (+を押してない) ❌ push が出ない → 未コミット ...

ディレクターが優先して覚えるべき VS Code 機能(重要度順)

  ① ディレクターが優先して覚えるべき VS Code 機能(重要度順) 1️⃣ ファイル・構造把握系(最重要) 「全体を瞬時に理解できる」ことがディレクターの価値 エクスプローラー フォルダ構成を一目で把握 /src /dist /assets /components などの役割理解 Ctrl(Cmd)+ P ファイル名検索(超重要) Ctrl(Cmd)+ Shift + F プロジェクト全体検索(文言修正・影響範囲確認) 👉 「この文言、どこで使われてる?」が即わかる 2️⃣ HTML / CSS / JS を読むための機能 書けなくても「読める・追える」ことが重要 シンタックスハイライト タグ/クラス/JS構文の色分け 折りたたみ(コードフォールディング) セクション単位で構造理解 Go to Definition(右クリック) CSSやJSの参照元を追える 👉 デザイナー・エンジニアとの会話が一段上がる 3️⃣ 差分・修正管理(地味だが超重要) 「何を変えたか説明できるディレクター」になる Git差分表示(赤・緑) 変更行の可視化 元に戻す(Undo / Discard Changes) 👉 「どこ触りました?」に即答できる 4️⃣ プレビュー・動作確認系 自分で一次チェックできるかどうか Live Server HTML/CSSの即時プレビュー ブラウザとの往復を減らす 👉 エンジニアの確認工数を減らせるディレクターは信頼される 5️⃣ マルチカーソル・一括編集 軽微修正を爆速で 複数カーソル 同時編集 同一単語の一括選択 👉 文言修正・class名変更が一瞬 6️⃣ 拡張機能(最低限) 入れすぎないのがディレクター流 Live Server Prettier(フォーマット統一) Japanese Language Pack GitLens(余裕があれば) ② ディレクター向け・おすすめ学習ステップ STEP1|「VS Codeに慣れる」(1〜2日) 目的:怖さをなくす フ...