Suggestion Change便利だけどちょっとやらかした

エンジニア雑記

GithubのSuggestion Change便利

プルリクエストでのフィードバックを取り込む - GitHub Docs

  • 簡単なtypoとか気軽に取り込んでもらえる
  • 取り込まなくてもコード例出しつつコメント書ける

めっちゃお気に入りでよく使ってるけど、ちょっと不便なケースに遭遇した

こんなPR

<?php

class Hoge extends TestCase
{
  use PHPUnit\Framework\TestCase;

  /**
    * @test
    */
  public function hogeTest()
  {
    $hoge_repository = DB::cretateRepository('App\Repository\HogeRepository');
    $data = $hoge_repository->getData(1);
    $this->assertNotNull($data);
  }
}

にこんなsuggestion

+$hoge_repository = DB::cretateRepository(HogeRepository::class);
-$hoge_repository = DB::cretateRepository('App\Repository\HogeRepository');
Class::class記法推奨ってsuggesionした

当然一行だけだしスッと取り込んでもらえた(SUTだけに)

どうなったか

CIでコケた

Error: Class 'App\Repository\HogeRepository' not found

なんでコケたか

Class::class記法はuse App\Repository\HogeRepository;する必要があった
一行の変更だしそりゃ当然漏れるよね

どうするか

  1. Suggestionはまあコメントのみにとどめて、基本手元で修正する
    IDE使ってたら自動でuseしてくれる
  2. テストコードならCIコケて分かるし気にしない
    そのためのCI？後そのためのslack通知？
    CI！SUT！Failedって感じで…